Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Rename dependency is_resolved to is_pinned #3888

Merged
merged 5 commits into from
Oct 20, 2024

Conversation

AyanSinhaMahapatra
Copy link
Member

@AyanSinhaMahapatra AyanSinhaMahapatra commented Aug 7, 2024

Renaming the dependency attribute is_resolved to is_pinned to more accurately represent this attribute accurately. This is more relevant after the changes in aboutcode-org/scancode.io#1066

Reference: aboutcode-org/scancode.io#1066

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)

Renaming the dependency attribute is_resolved to is_pinned to
more accurately represent this attribute accurately.
This is more relevant after the changes in aboutcode-org/scancode.io#1066

Reference: aboutcode-org/scancode.io#1066
Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
Also fix test failures.

Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
@AyanSinhaMahapatra AyanSinhaMahapatra force-pushed the rename_dependency_is_resolved branch from f8cd741 to 705630c Compare August 7, 2024 17:35
@pombredanne pombredanne changed the title Rename dependency attribute is_resolved Rename dependency is_resolved to is_pinned Aug 12, 2024
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.

Here are some nit for your consideration

@AyanSinhaMahapatra AyanSinhaMahapatra force-pushed the rename_dependency_is_resolved branch from 705630c to db38266 Compare August 12, 2024 13:27
Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
@AyanSinhaMahapatra AyanSinhaMahapatra force-pushed the rename_dependency_is_resolved branch from db38266 to caeb49f Compare October 19, 2024 11:02
Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
@AyanSinhaMahapatra AyanSinhaMahapatra merged commit 05b9b7f into develop Oct 20, 2024
39 checks passed
@AyanSinhaMahapatra AyanSinhaMahapatra deleted the rename_dependency_is_resolved branch October 20, 2024 12:35
@AyanSinhaMahapatra
Copy link
Member Author

Thanks @pombredanne, resolved your comments and merged!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants