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

chore(packaging): support version parsing in packaging v22 [backport #4752 to 1.6] #4765

Merged
merged 3 commits into from
Dec 12, 2022

Conversation

mabdinur
Copy link
Contributor

@mabdinur mabdinur commented Dec 9, 2022

Description

Backport #4752

Reviewer Checklist

  • Title is accurate.
  • Description motivates each change.
  • No unnecessary changes were introduced in this PR.
  • Avoid breaking API changes unless absolutely necessary.
  • Tests provided or description of manual testing performed is included in the code or PR.
  • Release note has been added and follows the library release note guidelines, or else changelog/no-changelog label added.
  • All relevant GitHub issues are correctly linked.
  • Backports are identified and tagged with Mergifyio.

## Description

Packaging v22 raises an InvalidVersion exception instead of returning a
LegacyVersion(). If an InvalidVersion is raised we should set the
packaging version to `0.0.0`

## Reviewer Checklist
- [ ] Title is accurate.
- [ ] Description motivates each change.
- [ ] No unnecessary changes were introduced in this PR.
- [ ] Avoid breaking
[API](https://ddtrace.readthedocs.io/en/stable/versioning.html#interfaces)
changes unless absolutely necessary.
- [ ] Tests provided or description of manual testing performed is
included in the code or PR.
- [ ] Release note has been added and follows the [library release note
guidelines](https://ddtrace.readthedocs.io/en/stable/contributing.html#Release-Note-Guidelines),
or else `changelog/no-changelog` label added.
- [ ] All relevant GitHub issues are correctly linked.
- [ ] Backports are identified and tagged with Mergifyio.
@mabdinur mabdinur added the changelog/no-changelog A changelog entry is not required for this PR. label Dec 9, 2022
@mabdinur mabdinur changed the title chore(packaging): support version parsing in packaging v22 [Backport #4752 to 1.6] chore(packaging): support version parsing in packaging v22 [backport #4752 to 1.6] Dec 9, 2022
@mabdinur mabdinur marked this pull request as ready for review December 9, 2022 17:27
@mabdinur mabdinur requested a review from a team as a code owner December 9, 2022 17:27
@brettlangdon brettlangdon enabled auto-merge (squash) December 9, 2022 19:05
@brettlangdon brettlangdon merged commit c5c5bec into 1.6 Dec 12, 2022
@brettlangdon brettlangdon deleted the backport-4752-1-6 branch December 12, 2022 20:14
@github-actions github-actions bot added this to the v1.6.4 milestone Dec 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog/no-changelog A changelog entry is not required for this PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants