Skip to content

Comments

pep440: support post/local for semver allows#158

Merged
sdispater merged 1 commit intopython-poetry:masterfrom
abn:fix/semver-allows
Apr 8, 2021
Merged

pep440: support post/local for semver allows#158
sdispater merged 1 commit intopython-poetry:masterfrom
abn:fix/semver-allows

Conversation

@abn
Copy link
Member

@abn abn commented Apr 3, 2021

This change ensures that post and local releases are taken into
consideration when checking if semver version instance allows
post and local build releases.

The following conditions now hold poetry.core.semver.Version.allows.

  • 3.0.0 allows 3.0.0+local.1, 3.0.0-1
  • 3.0.0+local.1 disallows 3.0.0+local.2, allows 3.0.0-1
  • 3.0.0-1 disallows 3.0.0, 3.0.0+local.1, allows 3.0.0-1+local.1

This is a follow up from #157 .

Note this does not change equality check behaviour.

@abn abn requested a review from a team April 3, 2021 00:52
@abn abn force-pushed the fix/semver-allows branch from 0ccfec4 to 6fa5484 Compare April 7, 2021 00:48
@abn abn requested a review from sdispater April 7, 2021 00:49
This change ensures that post and local releases are taken into
consideration when checking if semver version instance allows
post and local build releases.

The following conditions now hold `poetry.core.semver.Version.allows`.

- `3.0.0` allows `3.0.0+local.1`, `3.0.0-1`
- `3.0.0+local.1` disallows `3.0.0+local.2`, allows `3.0.0-1`
- `3.0.0-1` disallows ``3.0.0`, `3.0.0+local.1`, allows `3.0.0-1+local.1`
@abn abn force-pushed the fix/semver-allows branch from 6fa5484 to 364f025 Compare April 7, 2021 11:10
@sdispater sdispater merged commit 716375e into python-poetry:master Apr 8, 2021
@sdispater sdispater mentioned this pull request Apr 8, 2021
@abn abn deleted the fix/semver-allows branch April 8, 2021 14:34
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