-
Notifications
You must be signed in to change notification settings - Fork 82
Cleanup config and CI #214
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
Cleanup config and CI #214
Conversation
- improve job name - bump versions v3->v4 for actions/checkout and actions/upload-artifact - make artifact name unique (needed for action v4)
- Ubuntu-24.04 runner (ubuntu-latest as of Jan 2025) doesn't have it pre-installed
- add libreadline-dev
- remove unnecessary settings - create settings for synapps 5.8, 6.0 and 6.3 - use these settings in the ci-scripts-build - use reasonable defaults - make the default builds use dependency defaults
|
❌ Build asyn 1.0.286 failed (commit b5c3491063 by @ralphlange) |
|
❌ Build asyn 1.0.287 failed (commit 678c1c52de by @ralphlange) |
caaf25b to
52dda2b
Compare
|
❌ Build asyn 1.0.288 failed (commit 2c03bd23b8 by @ralphlange) |
- integrate features/changes from Base - use the new setup files - 'cinst' -> 'choco install' (Chocolatey dropped the alias)
52dda2b to
f6b6265
Compare
|
❌ Build asyn 1.0.289 failed (commit 8abff135ba by @ralphlange) |
|
@hjunkes @MarkRivers I think this is ready for review (and merge) now. |
|
❌ Build asyn 1.0.292 failed (commit 6e7cc2fb9a by @hjunkes) |
MarkRivers
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.
This looks good to me.
This PR tries to get the asyn CI into a more reasonable state.