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

Example to support / Push to multiple git remotes. #1784

Merged

Conversation

joshirohit100
Copy link
Contributor

@joshirohit100 joshirohit100 commented Aug 28, 2024

I have noticed about the confusion to push / use multiple git remotes the artifact.

This MR is adding an example/usage in command help for same.

Screenshot 2024-08-28 at 10 17 34 AM

Will this automatic generate / update doc https://docs.acquia.com/acquia-cloud-platform/add-ons/acquia-cli/commands/push:artifact or thats separate?

Copy link

codecov bot commented Aug 28, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.22%. Comparing base (c072606) to head (88ceab0).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #1784   +/-   ##
=========================================
  Coverage     92.21%   92.22%           
  Complexity     1817     1817           
=========================================
  Files           121      121           
  Lines          6812     6813    +1     
=========================================
+ Hits           6282     6283    +1     
  Misses          530      530           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

Try the dev build for this PR: https://acquia-cli.s3.amazonaws.com/build/pr/1784/acli.phar

curl -OL https://acquia-cli.s3.amazonaws.com/build/pr/1784/acli.phar
chmod +x acli.phar

@joshirohit100 joshirohit100 force-pushed the example_for_multiple_git_remotes branch from 8e76758 to dce0268 Compare August 28, 2024 10:44
@vyasamit2007
Copy link

This works for us on a actual proejct.

@danepowell
Copy link
Contributor

Thanks for the PR! I've never tried providing multiple URLs but I trust this works as you've described.

docs.acquia.com should automatically get updated when this is merged.

@danepowell danepowell merged commit 9960580 into acquia:main Sep 5, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants