magento / magento-cloud-docker

All Submissions you make to Magento Inc. (“Magento") through GitHub are subject to the following terms and conditions: (1) You grant Magento a perpetual, worldwide, non-exclusive, no charge, royalty free, irrevocable license under your applicable copyrights and patents to reproduce, prepare derivative works of, display, publically perform, sublicense and distribute any feedback, ideas, code, or other information (“Submission") you submit through GitHub. (2) Your Submission is an original work of authorship and you are the owner or are legally entitled to grant the license stated above. (3) You agree to the Contributor License Agreement found here: https://github.com/magento/magento2/blob/master/CONTRIBUTOR_LICENSE_AGREEMENT.html
Open Software License 3.0
256 stars 192 forks source link

Add built-in support for n98-Magerun2 in the deploy container #345

Open MagicLegend opened 2 years ago

MagicLegend commented 2 years ago

Is your feature request related to a problem? Please describe. Magerun2 solves some annoyances devs have to deal with when developing with Magento. Directly running cronjobs for example.

Describe the solution you'd like Would be great if the deploy container (so the cli php images) now used for magento commands would also support running n98-magerun2. For example:

docker-compose run --rm deploy magerun2 <command>

Note that magerun2 will also expose the regular Magento CLI, so it could also be used as a replacement of the magento-command command.

Describe alternatives you've considered Fixing the container myself locally. But that means my colleagues would have to go through the same steps as well. Changing it at the source would be a better solution IMO.

Additional context Magerun2 is under the MIT licence, so license wise I think it should be fine to distribute the Magerun binary directly in the container.

The original https://github.com/meanbee/docker-magento2 already had this exact functionality. Perhaps most (all?) of what is needed can be taken from that repository.