ci: Trigger mobile workflows on workflow change#24212
Conversation
Signed-off-by: Ryan Northey <ryan@synca.io>
f535fcb to
0941f9b
Compare
|
just to flag this is the kind of problem i was referring to with the " with bazel, you have a hermetic principle and if everything sticks to it you can trust your inputs and bazel automatically blows the cache if it detects any change in them with this type of pattern you are effectively managing the inputs manually which means that in all of the places you want to control program flow according to change you need to capture exactly the correct inputs/changes, and supposing you manage to do that, you then have to maintain them as different parts of the system change |
|
a similar example would be where the mobile part of the repo uses tooling from the envoy part of the repo - the tooling might change in the envoy part, which wont be detected as a mobile change and so wont get tested until the mobile part itself changes (we might check and see that it doesnt pose any problem, but there is not guarantee that it will continue to do so) |
alyssawilk
left a comment
There was a problem hiding this comment.
very good call, thanks!
If the workflow has changed but is not tested, it might break next time it actually is tested
Signed-off-by: Ryan Northey ryan@synca.io
Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]