-
Notifications
You must be signed in to change notification settings - Fork 31
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
renaming macos to Darwin #183
renaming macos to Darwin #183
Conversation
Signed-off-by: greg pereira <[email protected]>
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!
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.
Can you also change .github/workflows
please? These names are load bearing on the CI
This should not be changed, |
My appologies, seems I missunderstood what you were saying, I will update the filenames in the workflows. |
Signed-off-by: greg pereira <[email protected]>
e3e2500
to
f00b80a
Compare
/cc @mihaimaruseac @smeiklej Do tests require approval to run? Trying to test the workflow changes requested |
They do, yes. |
Signed-off-by: greg pereira <[email protected]>
5273505
to
a604f5a
Compare
Signed-off-by: greg pereira <[email protected]>
a604f5a
to
bf2a155
Compare
Ready for real Review |
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.
Thank you!
Addresses: #181
Summary
While working on #78, I quickly found that the issue is not so much speed but platform enablement for Darwin, a theme I am finding in the repo. This change will help us in bringing alignment between the scripts that are meant to work on both Linux and Darwin, by simpifying / hardneing our process around pieces that are platform dependent
Release Note
Change naming conventions from
macOS
toDarwin
Documentation
No changes in the sigstore dev docs, but the repo
README.md
has been updated