-
Notifications
You must be signed in to change notification settings - Fork 2
port CI changes #1030
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
port CI changes #1030
Conversation
Do we need the Cargo.lock changes? |
maybe not but since the main issue is sysinfo/ntapi not being updated maybe just keep, what other fixes did we need to port? |
Should this be merged into this branch? https://github.com/swift-nav/swift-toolbox/tree/swift-toolbox-v4.0.0-branch |
both branches are identical |
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.
Assuming there are changes to the CI repo that we'll need too? Do we need to branch the CI repo?
yep we do, currently here: there are some issues with mac signing which looks odd 🤔 |
https://github.com/swift-nav/swift-toolbox/releases/tag/v4.1.11 will need to merge this in to fix #1035 ci |
Frontend and Release Workflow Started here |
oops, didn't mean to close the PR |
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.
feel free to merge once we're able to get CI passing cleanly
Frontend and Release Workflow Started here |
Decoupled CI is working, just taking a bit longer, I think we adjust the timeout to 45 minutes it should pass cleanly: https://github.com/swift-nav/swift-toolbox-ci/actions/runs/4942658545 |
ports decoupled CI changes to v4.0.13 which is released at
https://www.swiftnav.com/resource-library?product=Swift+Console&category=Installer&release=Latest&search=Swift+Console&filters=no&title=Swift+Console