Skip to content

Improve cargo package detection support#3783

Merged
AyanSinhaMahapatra merged 3 commits intodevelopfrom
fix-cargo-workspaces
Jun 10, 2024
Merged

Improve cargo package detection support#3783
AyanSinhaMahapatra merged 3 commits intodevelopfrom
fix-cargo-workspaces

Conversation

@AyanSinhaMahapatra
Copy link
Member

@AyanSinhaMahapatra AyanSinhaMahapatra commented May 20, 2024 •
edited
Loading

Improvements and bugfixes:

  • Fix for parser crashing on cargo workspaces
  • Fix a bug in dependency parsing (we were not returning any dependencies)
  • Also support getting dependency versions from workspace
  • Support more attributes from cargo
  • Better handle workspace data thorugh extra_data ({workspace = true} would not be seen anymore even on parser failing)
  • clean up workspace data from extra data on resolution

Reference: #3769

Tasks

  • Reviewed contribution guidelines
  • PR is descriptively titled 📑 and links the original issue above 🔗
  • Tests pass -- look for a green checkbox ✔️ a few minutes after opening your PR
    Run tests locally to check for errors.
  • Commits are in uniquely-named feature branch and has no merge conflicts 📁
  • Updated documentation pages (if applicable)
  • Updated CHANGELOG.rst (if applicable)

@AyanSinhaMahapatra
Improve cargo workspace support
4b7a3c1 
Reference: #3769
Signed-off-by: Ayan Sinha Mahapatra <[email protected]>
@AyanSinhaMahapatra AyanSinhaMahapatra mentioned this pull request May 22, 2024
Closed
pombredanne
pombredanne approved these changes Jun 8, 2024
View reviewed changes
Copy link
Member

@pombredanne pombredanne left a comment

Choose a reason for hiding this comment

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

LGTM! great. Just need a changelog

@AyanSinhaMahapatra AyanSinhaMahapatra changed the title Improve cargo workspace support Improve cargo package detection support Jun 10, 2024
@AyanSinhaMahapatra AyanSinhaMahapatra linked an issue Jun 10, 2024 that may be closed by this pull request
Unexpected output for field vcs_url in cargo workkspaces #3769
Closed
@AyanSinhaMahapatra
Copy link
Member Author

AyanSinhaMahapatra commented Jun 10, 2024

Thanks @pombredanne , added CHANGELOG, now merging!

@AyanSinhaMahapatra AyanSinhaMahapatra merged commit 8f5daf7 into develop Jun 10, 2024
@AyanSinhaMahapatra AyanSinhaMahapatra deleted the fix-cargo-workspaces branch June 10, 2024 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pombredanne pombredanne pombredanne approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Unexpected output for field vcs_url in cargo workkspaces

2 participants

@AyanSinhaMahapatra @pombredanne