- Notifications
You must be signed in to change notification settings - Fork13.3k
Rework CI workflows#8688
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Rework CI workflows#8688
Uh oh!
There was an error while loading.Please reload this page.
Conversation
mcspr commentedOct 11, 2022
'Required' workflows never going to work with path filtering, though. Github suggests somethingvery weird with name substitution -https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/defining-the-mergeability-of-pull-requests/troubleshooting-required-status-checks#handling-skipped-but-required-checks - which also brings up the fact that workflows matchby name. ^ matched with Windows build, for some reason I think those are somewhere in the org settings, so need to edit that |
mcspr commentedOct 13, 2022
Experimenting with folds / groups, using them may result in almost unreadable error messages b/c Githubdefers log page loading. External example |
- split workflows into separate files to trigger by path this should help out documentation and boards / eboot / pkg files updates, since those *wont* trigger usual build stuff anymore- build*.sh whatever merged into just common.sh and build.sh trigger different parity builds, mod % rem and allow to set .ino list through the environment variable- removes unnecessary temporary files, try to use more pipes move remaining ones into cache dir instead of PWD- remove legacy TRAVIS env vars, use ESP8266_ARDUINO prefix for config- remove Windows path workarounds- hardware/ and ide/ directories are set through envionment do not force specific paths, simplify builds on local machine- sketch list is set through environment. expicit paths for Windows and macOS builders. platformio also gets a real shuffled list instead of mod and rem magic numbers- detect root of the repo through git cli, not base{name,dir} or relative paths- split workflows into separate files to trigger by path this should help out documentation and boards / eboot / pkg files updates, since those *wont* trigger usual build stuff anymore- build*.sh whatever merged into just common.sh and build.sh trigger different parity builds, mod % rem and allow to set .ino list through the environment variable- removes unnecessary temporary files, try to use more pipes move remaining ones into cache dir instead of PWD- remove legacy TRAVIS env vars, use ESP8266_ARDUINO prefix for config- remove Windows path workarounds- hardware/ and ide/ directories are set through envionment do not force specific paths, simplify builds on local machine- sketch list is set through environment. expicit paths for Windows and macOS builders. platformio also gets a real shuffled list instead of mod and rem magic numbers- detect root of the repo through git cli, not base{name,dir} or relative paths
Uh oh!
There was an error while loading.Please reload this page.
this should help out documentation and boards / eboot / pkg files updates, since thosewont trigger usual build stuff anymore
https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#onpushpull_requestpull_request_targetpathspaths-ignore
trigger different parity builds, mod % rem and allow to set .ino list through the environment variable