-
Notifications
You must be signed in to change notification settings - Fork 29.7k
add test to verify that nothing changes in the git workspace #77100
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
|
cc @godofredoc @CaseyHillers @christopherfujino how do i enable something like this for the LUCI shards as well? |
|
@Hixie should this be run on all of the Flutter checks? Would it make sense to add this to bots/test.dart and run it at the end of every recipe? |
|
|
|
It's not, but we could add it to the missing builds. That test could be run at the end of every |
|
If we're adding something, easier to just add |
|
Yes, you'll need to add steps to the recipes. You can run the git command as a shell step in those recipes until we support go/flutter-single-recipe-builds The recipes repo is hosted on git-on-borg at https://flutter.googlesource.com/recipes/ |
|
We have been calling these use case ad hoc validations and we have been implementing them in the following way:
If this is something that won't run as an independent task and instead we want to run it on every task we will need to add it as a step to the relevant recipes: |
|
The idea is to run it at the end of every job that ran anything, to make sure the test didn't affect the git workspace. In any case it looks like it might be simpler to just wait until the recipes and such aren't written in python in another repo... I'm a little worried that for example if I made the change here it would catch a bug and break everything (unless there's a way to test the recipe changes ahead of landing them?). |
Recipe changes are tested on presubmit using builds from framework, devicelab hostonly and engine. Here is an example: https://ci.chromium.org/p/flutter/builders/try/recipes-with-led/b8853549532636159696? |
|
I've filed #78265 for now, going to close this. I'm unlikely to get to this in the near future unfortunately. |
Pre-launch Checklist
///).If you need help, consider asking for advice on the #hackers-new channel on Discord.