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

Only run publish workflow on release #8

Merged
merged 1 commit into from
Aug 8, 2024
Merged

Conversation

mrtz-j
Copy link
Contributor

@mrtz-j mrtz-j commented Aug 7, 2024

Description

This PR updates the publish.yml workflow to run only when a new release is published. I've removed the triggers for pull requests and pushes to the main branch, focusing the workflow solely on the release process.

Note: With this change, someone with the necessary permissions will need to create a new tag or release to trigger the publish workflow. This ensures that we only publish new versions of Giraffe.OpenApi when explicitly releasing a new version.

@nojaf
Copy link
Contributor

nojaf commented Aug 8, 2024

I'm going to give this a shot. As far as I'm understanding it I need to push a 0.0.1 tag and the action will do the rest.

This is fine for now, but in the future we should really automate this and publish/release once the changelog in the main branch has changed. (example of how we did that for the ionide analyzers, the tl;dr is that we extract the changelog version via https://github.com/ionide/KeepAChangelog and just compare with previous version)

@nojaf nojaf merged commit 68b6656 into giraffe-fsharp:main Aug 8, 2024
4 checks passed
@mrtz-j
Copy link
Contributor Author

mrtz-j commented Aug 8, 2024

Sounds like a good idea. I'll give it a try next weekend. 👍

@nojaf
Copy link
Contributor

nojaf commented Aug 8, 2024

Cool, tremendous thanks for all your work on this and for letting us have this repository in this organization!

The NuGet package was published to https://www.nuget.org/packages/Giraffe.OpenApi

//cc @dungpa

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.

2 participants