Skip to content

Conversation

@deborahgu
Copy link
Member

@deborahgu deborahgu commented Feb 20, 2025

pull-all-repos-in-directory enhancements:

  • -q flag to make it less noisy
  • -c flag so that it will do garbage collection and committed branch cleanup
  • -d flag so you can specify a top-level directory to run in

without any of these flags, behavior should be unchanged.


I've completed each of the following or determined they are not applicable:

  • Made a plan to communicate any major developer interface changes (or N/A)

pull-all-repos-in-directory enhancements:
* -q flag to make it less noisy
* -c flag so that it will do garbage collection and committed branch cleanup
* -d flag  so you can specify a top-level directory to run in

without any of these flags, behavior should be unchanged.
Copy link
Contributor

@justinhynes justinhynes left a comment

Choose a reason for hiding this comment

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

FWIW -- I'm not very adept at shell scripting, but I pulled and tested it and seems good! It looks like it did what was desired/expected and I like the cleanup options.

One nit -- I had to chmod +x the script to make it executable once I pulled down your branch. Can we update the permissions before merge?

JH  pointed out that the file permissions should be 755.
@deborahgu deborahgu merged commit 3b24510 into master Feb 25, 2025
14 checks passed
@deborahgu deborahgu deleted the dkaplan1/pull-all-repos-enhancements branch February 25, 2025 15:48
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