-
Notifications
You must be signed in to change notification settings - Fork 22
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
[Issue #2329] API-CI/CD Automation Conflict #2520
Conversation
@babebe - if you make a change to both the API folder and DB models in the same commit, does this work in parallel with your change? (And good find to look at the concurrency, I hadn't even thought of that when writing up the ticket) |
7941ab5
to
373ce85
Compare
Please make sure to follow the PR naming conventions before you wrap up: https://github.com/HHS/simpler-grants-gov/blob/main/DEVELOPMENT.md#title , thanks. |
Yes i will definitely do that |
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.
probably wait for @chouinar though
Summary
Fixes #{2329}
Time to review: 5 mins
Changes proposed
Merge workflows into one file.
Context for reviewers
Updates to PR while running the jobs(ex. ERD diagram generated) would cancel the other job. Due to jobs being similar opted for running it under one action.
Additional information