-
Notifications
You must be signed in to change notification settings - Fork 230
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
Create a GitHub action to create a release #821
Conversation
b9f24de
to
7859165
Compare
This should replace the script that I have locally to test swift-format for a release and at the same time expand the configurations that are being tested.
7859165
to
a29f7e6
Compare
runs-on: ubuntu-latest | ||
steps: | ||
- run: | | ||
if [[ "${{ github.triggering_actor }}" != "ahoppen" ]]; then |
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.
We should create a team for swift-format release, so it's not blocked on one person.
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.
My thought was that it’s pretty easy to modify the workflow to shift responsibilities. That way the responsibility transfer is explicit.
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.
Overall looks good, just one suggestion on using team to avoid being blocked on one person. You might want to add @bnbarham.
This should replace the script that I have locally to test swift-format for a release and at the same time expand the configurations that are being tested.
A sample run can be seen here: https://github.com/ahoppen/swift-format/actions/runs/11089874502