Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add bin/ece-patches command #1171

Merged
merged 4 commits into from
Jul 31, 2024
Merged

Conversation

DmitryFurs
Copy link
Contributor

Summary

This PR adds a new command bin/ece-patches to enable running Adobe Commerce Cloud patches tool commands from the host inside the container. This addition aims to simplify the process of applying patches by providing a straightforward command.

Usage

To use the new alias, run the following command from the host:

bin/ece-patches apply

image

This command will execute the ./vendor/bin/ece-patches inside the container, allowing you to apply patches as documented in the Adobe Commerce Cloud Service documentation.

Copy link

what-the-diff bot commented Jun 7, 2024

PR Summary

  • Implementation of ECE Patches Checking Tool
    Added a new file named bin/ece-patches which contains a script. What this does is to validate if the ece-patches tool is already installed in the system and, if it is, to execute it using the bin/cli command.

  • README Documentation Update
    The README.md file has been updated to include our newly added command (bin/ece-patches) in the list of commands that users can run on our application. This provides guidance on command availability and usage to end users.

  • Inclusion of New Command in Makefile
    Made an update to the Makefile found in the compose directory. The modification involves including the recently implemented ece-patches command in the help menu, so when users explore the command options, they see this newly introduced functionality as an available choice.

@markshust markshust merged commit 83d238a into markshust:release/next Jul 31, 2024
1 check passed
@markshust
Copy link
Owner

Appreciate it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants