Skip to content

Improve version range resolution - #183

Merged
JonoYang merged 8 commits into
mainfrom
improve-vers-resolution
Sep 12, 2023
Merged

Improve version range resolution#183
JonoYang merged 8 commits into
mainfrom
improve-vers-resolution

Conversation

@keshav-space

@keshav-space keshav-space commented Sep 6, 2023

Copy link
Copy Markdown
Member
  • Handle invalid constraints sequence exception while testing whether a given version is present in given vers expression
  • Handle potential invalid version string
  • Skip vers resolution for unsupported ecosystems, currently on_demand_queue only supports maven and npm
  • Bump univers to v30.10.1

- issue aboutcode-org/univers#118

Signed-off-by: Keshav Priyadarshi <git@keshav.space>
Signed-off-by: Keshav Priyadarshi <git@keshav.space>
Signed-off-by: Keshav Priyadarshi <git@keshav.space>
Signed-off-by: Keshav Priyadarshi <git@keshav.space>

@JonoYang JonoYang 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.

@keshav-space I have some questions about the try-except blocks you've added.

Comment thread packagedb/api.py Outdated
Comment thread packagedb/api.py Outdated
Comment thread packagedb/api.py Outdated
Comment thread packagedb/api.py Outdated
Comment thread packagedb/api.py Outdated
Comment thread packagedb/api.py
Signed-off-by: Keshav Priyadarshi <git@keshav.space>
keshav-space and others added 3 commits September 12, 2023 14:49
Signed-off-by: Keshav Priyadarshi <git@keshav.space>
Signed-off-by: Keshav Priyadarshi <git@keshav.space>
Signed-off-by: Jono Yang <jyang@nexb.com>
@JonoYang

Copy link
Copy Markdown
Member

@keshav-space Thanks for the univers update! I've regenerated a test file, so we can pass all the tests. Merging.

@JonoYang
JonoYang merged commit 8428cb8 into main Sep 12, 2023
@keshav-space
keshav-space deleted the improve-vers-resolution branch September 12, 2023 17:08
JonoYang added a commit that referenced this pull request Apr 14, 2025
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.

3 participants