Skip to content

add native github ranges support - #49

Merged
pombredanne merged 1 commit into
aboutcode-org:mainfrom
TG1999:version_ranges/native_github
Mar 16, 2022
Merged

add native github ranges support#49
pombredanne merged 1 commit into
aboutcode-org:mainfrom
TG1999:version_ranges/native_github

Conversation

@TG1999

@TG1999 TG1999 commented Mar 16, 2022

Copy link
Copy Markdown
Contributor

Signed-off-by: Tushar Goel tushar.goel.dav@gmail.com

@TG1999
TG1999 force-pushed the version_ranges/native_github branch 8 times, most recently from 5227a3d to e717dd6 Compare March 16, 2022 13:28
Comment thread src/univers/version_range.py Outdated
>>> vr = build_constraint_from_github_advisory_string("pypi","< 9.0")
>>> assert str(vr) == "<9.0", str(vr)
"""
vrc = RANGE_CLASS_BY_SCHEMES.get(scheme)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

IMHO we can be sure that scheme exists:

Suggested change
vrc = RANGE_CLASS_BY_SCHEMES.get(scheme)
vrc = RANGE_CLASS_BY_SCHEMES[scheme]

@TG1999
TG1999 force-pushed the version_ranges/native_github branch 2 times, most recently from 0872494 to 984204a Compare March 16, 2022 13:38

@pombredanne pombredanne left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!

@TG1999
TG1999 force-pushed the version_ranges/native_github branch from 984204a to ac62daf Compare March 16, 2022 13:40
Signed-off-by: Tushar Goel <tushar.goel.dav@gmail.com>
@TG1999
TG1999 force-pushed the version_ranges/native_github branch from ac62daf to 1ad6439 Compare March 16, 2022 13:42
@pombredanne
pombredanne merged commit 737f34a into aboutcode-org:main Mar 16, 2022
@pombredanne

Copy link
Copy Markdown
Member

Thanks!
@TG1999 I am sending you an invite to get commit rights here

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