Skip to content
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

Merged
merged 4 commits into from
May 16, 2024

Conversation

Gregory-Pereira
Copy link
Contributor

@Gregory-Pereira Gregory-Pereira commented May 16, 2024

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 to Darwin

Documentation

No changes in the sigstore dev docs, but the repo README.md has been updated

Signed-off-by: greg pereira <[email protected]>
@Gregory-Pereira Gregory-Pereira requested a review from a team as a code owner May 16, 2024 15:54
smeiklej
smeiklej previously approved these changes May 16, 2024
Copy link
Collaborator

@smeiklej smeiklej left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Copy link
Collaborator

@mihaimaruseac mihaimaruseac left a 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

@Gregory-Pereira
Copy link
Contributor Author

Gregory-Pereira commented May 16, 2024

Can you also change .github/workflows please? These names are load bearing on the CI

This should not be changed, macos-latest is the name of the runner, and means something to github actions. If I change this to darwin the runner will not pick it up. See Github actions documentation around using github hosted runners. This intricacy is also documented in the original issue.

@Gregory-Pereira
Copy link
Contributor Author

My appologies, seems I missunderstood what you were saying, I will update the filenames in the workflows.

@Gregory-Pereira
Copy link
Contributor Author

Gregory-Pereira commented May 16, 2024

/cc @mihaimaruseac @smeiklej Do tests require approval to run? Trying to test the workflow changes requested

@smeiklej
Copy link
Collaborator

They do, yes.

@Gregory-Pereira Gregory-Pereira force-pushed the swap-macOS-for-Darwin branch 7 times, most recently from 5273505 to a604f5a Compare May 16, 2024 18:08
Signed-off-by: greg pereira <[email protected]>
@Gregory-Pereira
Copy link
Contributor Author

Ready for real Review

Copy link
Collaborator

@mihaimaruseac mihaimaruseac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@mihaimaruseac mihaimaruseac merged commit 66c46cb into sigstore:main May 16, 2024
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants