-
Notifications
You must be signed in to change notification settings - Fork 29.7k
Make the pub roller bot re-generate gradle lockfiles #149355
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
Conversation
|
(triage): I spoke to @gmackall last week and he says this is still on his radar. |
e648095 to
70cc584
Compare
|
@gmackall FYI, I updated this branch. Turned into somewhat of a yak shave, but I updated generate_gradle_lockfiles.dart to no longer require the android directories via stdin (since that would have been awkward to pipe from the packages_autoroller.dart script) and just taught it to discover these all itself. |
Wow, thanks for taking the time to do that! This looks great to me, but I can't technically approve because it is my PR |
christopherfujino
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Fixes #142475
Pre-launch Checklist
///).If you need help, consider asking for advice on the #hackers-new channel on Discord.