Skip to content

Conversation

alok1304
Copy link
Contributor

@alok1304 alok1304 commented Dec 30, 2024

Renaming the dependency attribute is_resolved to is_pinned. Output changes when we use is_resolved.

Reference: #634

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
  • Looked for possible updates in documentation and added updates if
    applicable
  • Updated CHANGELOG.rst

Signed-off-by: Alok Kumar [email protected]

@alok1304 alok1304 force-pushed the rename_dependency_is_resolved branch from ff2a84b to 8568d5e Compare December 30, 2024 18:15
@OmkarPh
Copy link
Collaborator

OmkarPh commented Jan 11, 2025

Hey @alok1304 , sorry for the delayed response

Can you also rename resolved to pinned in labels and UI fields too (eg. headerName in DependencyEntity.tsx)
Some test json files still have is_resolved, please change those (eg. minimal.json)
Also, update the output_format_version to the latest 4.0.0 in the test files

Thanks

Renaming the dependency attribute is_resolved to is_pinned. Output changes when we use is_resolved.

Reference: aboutcode-org#634
Signed-off-by: Alok Kumar <[email protected]>
@alok1304 alok1304 force-pushed the rename_dependency_is_resolved branch from a06679a to e967f9e Compare January 11, 2025 11:05
@alok1304
Copy link
Contributor Author

Hii @OmkarPh , Also I changed is_resolved to is_pinned and output_format_version is 4.0.0 in samples file also.
Review this PR, also tell any further modification.

@OmkarPh
Copy link
Collaborator

OmkarPh commented Jan 17, 2025

LGTM, thanks !

@OmkarPh OmkarPh merged commit 4811dac into aboutcode-org:develop Jan 17, 2025
6 checks passed
@alok1304 alok1304 deleted the rename_dependency_is_resolved branch January 20, 2025 21:22
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