Add update packages roller #100982
Add update packages roller #100982
Conversation
flutter-dashboard
bot
added
the
team
christopherfujino
force-pushed
the
add-update-packages-roller
branch
2 times, most recently
from
74cfcb7
to
c88aeb7
Compare
christopherfujino
force-pushed
the
add-update-packages-roller
branch
from
c88aeb7
to
be4dea5
Compare
christopherfujino
force-pushed
the
add-update-packages-roller
branch
from
4fc9a86
to
c7beb18
Compare
| static const String hostname = 'github.com'; | ||
|
|
||
| static const String prBody = ''' | ||
| This PR was generated by `flutter update-packages --force-upgrade`. |
There was a problem hiding this comment.
Is Flutter update-packages going to call this conductor logic?
There was a problem hiding this comment.
The flow will be:
luci recipe -> dart packages_autoroller.dart -> flutter update-packages --force-upgrade
There was a problem hiding this comment.
I'm using the conductor codebase because it has abstractions for cloning the flutter/flutter repo in a somewhat hermetic environment, and there's both unit test and integration test infrastructure already set up.
|
|
||
| Future<void> updatePackages({ | ||
| bool verbose = true, | ||
| String author = 'flutter-packages-autoroller <flutter-packages-autoroller@google.com>' |
There was a problem hiding this comment.
Is this a google group?
There was a problem hiding this comment.
Yes, the only member of which is the tools team.
There was a problem hiding this comment.
I don't imagine anyone will be grepping the commit message and then emailing the address other than bots and recruiters though.
First part of #106371
Once this merges, it will still not be productionized. Next steps will be to create a builder config, a recipe, and add it to .ci.yaml.