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

feat(workflow): version and tag update #6

Merged
merged 4 commits into from
Mar 23, 2024
Merged

Conversation

CoderJoshDK
Copy link
Contributor

feat(mirror): remove v from version

As mentioned in #5, uv does not use v in its version tagging. Considering that this repo is primarily a mirror of the uv repo, it should match.

feat(workflow): auto release on tag creation

Although this functionality is not required, some users might be confused if the tag and release version are not the same. This in reference to #4

Additionally, the body of the release points to the uv release

As mentioned in #5,
`uv` does not use `v` in its version tagging. Considering that this repo
is primarily a mirror of the `uv` repo, it should match.
Although this functionality is not required, some users might be
confused if the tag and release version are not the same. This in
reference to #4

Additionally, the body of the release points to the uv release
In relation to : #6 (comment)

Although not strictly necessary, it is good to follow "best practices." A release tag should only contain `\d+\.\d+\.\d+.*`
Copy link
Member

@zanieb zanieb left a comment

Choose a reason for hiding this comment

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

Thanks!

@zanieb zanieb merged commit 4da9ea9 into astral-sh:main Mar 23, 2024
charliermarsh pushed a commit to astral-sh/ruff-pre-commit that referenced this pull request Mar 30, 2024
## Summary

Add a GitHub workflow for creating a new release, automatically, when a
new tag is created.

This solves #78. The code is very similar to the changes added in
astral-sh/uv-pre-commit#6

The primary difference, is that the URL points to ruff and this one
checks for if the tag starts with a `v`, but it is optional. This lets
you change ruff versions to not have a `v` in the future. Although the
`mirror.py` will need to be updated.
Speaking of updates to `mirror.py`, I ran `ruff` on it and updated the
`subprocess` to raise an error on failure.

## Test Plan

Locally tested.
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.

None yet

2 participants