-
-
Notifications
You must be signed in to change notification settings - Fork 4.5k
Updating the test duration workflow #13170
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
silv-io
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! Will merge after the release
|
Maybe we should investigate the conflict here as well before merging /cc @k-a-il |
bentsku
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.
Seems like there is still a git conflict to resolve, otherwise the changes looks good 👍 thanks to the simplification you've made!
cc2d0b1 to
6810520
Compare
6810520 to
126a50b
Compare
Co-authored-by: Anastasia Dusak <[email protected]>
Motivation
The update-test-durations workflow logic now lives in the
metarepo as a reusable workflow.localstack/meta#29
This PR updates
localstackto call into the centralized version, reducing duplication and ensuring consistency across community and pro.Changes
.github/workflows/update-test-durations.yml.localstack/meta/.github/workflows/update-test-durations.yml.