-
Notifications
You must be signed in to change notification settings - Fork 7
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
Add workflow for update api spec files to main #73
Conversation
.github/workflows/update-data.yml
Outdated
|
||
- name: "List requirements" | ||
run: | | ||
cat requirements_system.txt |
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.
Is it system or ubuntu?
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.
its ubuntu, thanks
.github/workflows/update-data.yml
Outdated
|
||
- name: "Install System dependencies" | ||
run: | | ||
xargs sudo apt -y install < requirements_ubuntu.txt |
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.
Ideally we call apt install only once, but xargs will at least do batches, right?
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.
Not sure. Tried a different approach with tr
.
6eb5eca
to
64675c3
Compare
64675c3
to
122050b
Compare
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.
Thanks fro trying to avoid all the yaml weirdness with quotes.
I've added this data-update workflow on the docs-update branch and merged it (#66).
This workflow should really be on the default branch to work.