-
Notifications
You must be signed in to change notification settings - Fork 1
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
[ITG-79] Setup CI #15
Conversation
.github/workflows/bode_ci.yaml
Outdated
on: | ||
pull_request: | ||
paths: | ||
- "bode/**" | ||
branches: [develop, master] |
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.
Wdyt of adding:
push:
branches: [develop, master]
Since our repo is public, we don't have limitations on the CI
See Github pricing: https://github.com/pricing
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.
Well, why not 💸 💸 💸
I think we can go even further and do
on:
pull_request:
paths:
- "cabra/**"
push:
paths:
- "cabra/**"
.github/workflows/cabra_ci.yaml
Outdated
on: | ||
pull_request: | ||
paths: | ||
- "cabra/**" | ||
branches: [develop, master] |
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.
I would add it here as well.
push:
branches: [develop, master]
push: | ||
paths: | ||
- "bode/**" |
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.
I think every push means also running after pushing from local -- it may be doubled and unnecessary. Wdyt?
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.
Yeah, this way you can see if CI is passing without creating a PR, the feedback loop is a bit shorter. I think we can leave it like that, merge, observe if we have redundant workflow runs and then adjust accordingly if needed
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.
ok
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.
<3
CI setup for
bode
andcabra
.Resources
https://kulawska.atlassian.net/browse/ITG-79