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

DM-44453: Fix handling of release versions in v2 API. #130

Merged
merged 8 commits into from
May 23, 2024

Conversation

jonathansick
Copy link
Member

@jonathansick jonathansick commented May 21, 2024

  • The release kind is now being automatically set based on introspection of the tracking git ref when an edition is automatically created from a build.
  • The release kind is now included in the edition response models, and can be set or reset in the POST/PATCH models for an edition.
  • The lsst_doc tracking mode now accepts semver tags, not just the vX.Y tags typical of Rubin documentation releases.

A minimal update to get Pre-commit running again. isort needed updating
to work with the current pre-commit.
- Accept version tags that don't start with v.
- Allow an optional patch component to support full semver versions.
@jonathansick jonathansick force-pushed the tickets/DM-44453 branch 2 times, most recently from 32d2d20 to 1cf5097 Compare May 21, 2024 19:50
There's an issue building uwsgi that's stopping the type checking, but
doesn't affect the pytest environments?

Including build tools (build-essential) doesn't seem to help with
building uwsgi.

Also stop pushing to Docker Hub
The POST and PATCH endpoints for editions now allow the kind to be set.
If it is the __main slug, then the edition is always a "main" kind.
Otherwise inspect the tracking git_ref to set release/major/minor kinds.
Otherwise the edition is a draft.
@jonathansick jonathansick marked this pull request as ready for review May 23, 2024 19:27
@jonathansick jonathansick merged commit 2a0cd86 into main May 23, 2024
5 checks passed
@jonathansick jonathansick deleted the tickets/DM-44453 branch May 23, 2024 19:27
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.

1 participant