Skip to content

Conversation

@1nv0k32
Copy link
Collaborator

@1nv0k32 1nv0k32 commented Jul 13, 2023

ACTIVATE TAG PROTECTION FOR ALL TAGS

With this PR, we are creating:

  • actual releases with pushing a tag in vN.N.N format. -> version format: 1.3.3
  • test releases with main commit pushes -> version format: 1.3.4.dev3+gc543f61

Licensing issue is fixed.

Closes #618

@1nv0k32 1nv0k32 changed the title New versioning and fix for License WIP: New versioning and fix for License Jul 13, 2023
@1nv0k32 1nv0k32 force-pushed the bugfix/hatch-migration-fixes branch from c543f61 to 029af19 Compare July 13, 2023 21:52
@1nv0k32 1nv0k32 changed the title WIP: New versioning and fix for License New versioning and fix for License Jul 13, 2023
@1nv0k32 1nv0k32 requested a review from staticdev July 13, 2023 22:37
on:
push:
branches:
- main
Copy link
Owner

Choose a reason for hiding this comment

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

@1nv0k32 isn't this inverted? The release-dev.yml us the tag branches and the release.yml uses branch main?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It's the same behaviour as it was before. When you merge something into main, if there was no tag, you were considering it as a dev release, and if you were tagging main, you were considering it as a release. I've done the same, but in two separate files.


[build-system]
requires = ["hatchling"]
requires = ["hatchling", "hatch-vcs"]
Copy link
Owner

Choose a reason for hiding this comment

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

Why do we need hatch-vcs? Can't we just continue having the version on pyproject.toml and avoid an extra dependency?

It is also possible to have dynamic version like this with hatch alone.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I thought it's cooler and easier to maintain.
But if you prefer the regex method, I can change it back.

Copy link
Owner

Choose a reason for hiding this comment

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

It is OK, let's try!

@staticdev staticdev merged commit b636ac6 into staticdev:main Jul 17, 2023
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.

Broken release workflow after change to hatch

2 participants