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

Print manual push instructions #51

Merged
merged 4 commits into from
May 11, 2022
Merged

Conversation

mattwynne
Copy link
Member

@mattwynne mattwynne commented May 5, 2022

🤔 What's changed?

  • Output manual push instructions when --no-git-push option is used.

⚡️ What's your motivation?

Make it safer for people to test the tool by using the --no-git-push switch, but still reduce the amount of thinking they have to do to a minimum.

Works towards #48.

🏷️ What kind of change is this?

  • ⚡ New feature (non-breaking change which adds new behaviour)

📋 Checklist:


This text was originally generated from a template, then edited by hand. You can modify the template here.

@mattwynne mattwynne mentioned this pull request May 5, 2022
3 tasks
Base automatically changed from fix-tests to main May 5, 2022 17:43
@mattwynne mattwynne force-pushed the print-manual-push-instructions branch 3 times, most recently from e2827ba to cdc140a Compare May 5, 2022 17:54
Copy link
Contributor

@aurelien-reeves aurelien-reeves left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good :)

polyglot-release Outdated Show resolved Hide resolved
@mattwynne mattwynne force-pushed the print-manual-push-instructions branch from 49fa851 to 79d0467 Compare May 11, 2022 00:41
@mattwynne mattwynne force-pushed the print-manual-push-instructions branch 2 times, most recently from af0cc88 to 44b941b Compare May 11, 2022 04:21
Copy link
Contributor

@mpkorstanje mpkorstanje left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great!

@mattwynne mattwynne force-pushed the print-manual-push-instructions branch from 44b941b to 4af9565 Compare May 11, 2022 17:04
@mattwynne mattwynne merged commit a97a81a into main May 11, 2022
@mattwynne mattwynne deleted the print-manual-push-instructions branch May 11, 2022 17:05
@mpkorstanje mpkorstanje added this to the v1 milestone May 14, 2022
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