-
-
Notifications
You must be signed in to change notification settings - Fork 145
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
Some cosmetic changes to the workflow names #445
Conversation
name: Linux Python build | ||
name: linux |
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.
Shortening these so we can see more of the target in the CI summary
push: | ||
branches: [main] |
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 check ran twice per commit
d62178b
to
d17fff5
Compare
name: MacOS Python build | ||
name: macos |
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.
Since it's "darwin" in the triple I'm sort of on the fence here. I could go either way.
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.
Perhaps the idea with apple.yml
is that we'd be doing iOS, etc. cross-compiles? We could revisit at that point though.
d17fff5
to
accb28c
Compare
No description provided.