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

feat: add push command #2

Merged
merged 2 commits into from
Nov 10, 2023
Merged

feat: add push command #2

merged 2 commits into from
Nov 10, 2023

Conversation

antonag32
Copy link
Owner

This is bad practice which results in spaghetti code down the line. Moving flag parsing to the actual command execution is cleaner.

See: spf13/cobra#553

@antonag32 antonag32 force-pushed the mass-merge branch 5 times, most recently from d5ed0a9 to 755b673 Compare November 7, 2023 04:40
This is bad practice which results in spaghetti code down the line.
Moving flag parsing to the actual command execution is cleaner.

See: spf13/cobra#553
This new command lets users perform arbitrary actions on mass to a set
of repositories and push the changes upstream.
@antonag32 antonag32 changed the title fix: don't parse flags in init feat: add push command Nov 10, 2023
@antonag32
Copy link
Owner Author

Aditionally the new push command has been added

@antonag32 antonag32 merged commit e3e37b0 into main Nov 10, 2023
1 check passed
@antonag32 antonag32 deleted the mass-merge branch November 10, 2023 02:54
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.

1 participant