Skip to content

Conversation

@AyanSinhaMahapatra
Copy link
Member

@AyanSinhaMahapatra AyanSinhaMahapatra commented Oct 21, 2025

  • Do not set flag_value as None in click Option

    From click 8.3.0 flag_value is set as a sentinel
    value UNSET instead of None, and since we maintain a PluggableCommanLineOption subclass, we need to
    maintain the same default values for an Option to
    avoid issues.

  • Also drops python 3.9 support and adds python 3.14 support.

Reference: https://github.com/pallets/click/releases/tag/8.3.0
Reference:

From click 8.3.0 flag_value is set as a sentinel
value UNSET instead of None, and since we maintain
a PluggableCommanLineOption subclass, we need to
maintain the same default values for an Option to
avoid issues.

Reference: https://github.com/pallets/click/releases/tag/8.3.0
Reference: aboutcode-org/scancode-toolkit#4572
Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
This removes a warning seen in initial virtualenv configuration.

Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
Reference: aboutcode-org/scancode-toolkit#4369
Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
Remove special file '6-FIFOTYPE' from types.tar to fix test failure
as in py3.14 tarfiles extraction fails with tarfile.SpecialFileError
for special files.

Reference: #88
Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
@AyanSinhaMahapatra AyanSinhaMahapatra changed the title Do not set flag_value as None in click Option Fix click compatibility and update python support Oct 22, 2025
Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
Copy link
Member

@JonoYang JonoYang left a comment

Choose a reason for hiding this comment

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

LGTM, merging!

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