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

Add major version updater workflow #27

Merged
merged 12 commits into from
Jun 6, 2023

Conversation

ahatzz11
Copy link
Contributor

@ahatzz11 ahatzz11 commented Jun 6, 2023

This PR adds support for creating/overriding a major version when a release is made. This allows people to have a single reference that does not need to be updated in order to get new non-breaking stuff.

Before:

 uses: github/[email protected]

After:

 uses: github/stale-repos@v1

Fixes #26

@ahatzz11 ahatzz11 marked this pull request as ready for review June 6, 2023 17:55
@ahatzz11 ahatzz11 requested a review from zkoppert as a code owner June 6, 2023 17:55
echo tag=${tag}
echo "version=${version}
echo "major=${major}"
} >> "$GITHUB_OUTPUT"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@zkoppert The linter forced this syntax, but I personally think this is a case of 'linter makes things less-common and more annoying', vs:

echo "tag=${tag}" >> "$GITHUB_OUTPUT"
echo "version=${version}" >> "$GITHUB_OUTPUT"
echo "major=${major}" >> "$GITHUB_OUTPUT"

Happy to keep it as is but thought I'd call it out.

Copy link
Member

Choose a reason for hiding this comment

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

I agree with you here, I'm disabling that check and reverting to original syntax...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nice! Appreciate the change 👍

Copy link
Member

Choose a reason for hiding this comment

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

Looks like I might not have the rc file in the correct place but i'll get there. 😅

@zkoppert zkoppert added the automation automation label Jun 6, 2023
@zkoppert zkoppert merged commit 8b516b7 into github:main Jun 6, 2023
@ahatzz11 ahatzz11 deleted the major-version-updater branch June 7, 2023 12:56
@ahatzz11
Copy link
Contributor Author

ahatzz11 commented Jun 7, 2023

@zkoppert I like how this ended with just getting rid of the linter 😂 Thanks for the assist

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automation automation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Major version is not tagged
2 participants