Skip to content

Fixed licensing dedup issue - #128

Merged
AyanSinhaMahapatra merged 20 commits into
mainfrom
67_dedup_issue
May 15, 2026
Merged

Fixed licensing dedup issue#128
AyanSinhaMahapatra merged 20 commits into
mainfrom
67_dedup_issue

Conversation

@chinyeungli

@chinyeunglichinyeungli commented Mar 16, 2026

Copy link
Copy Markdown
Contributor

Ref #67

This PR fixes the dedup issue with flattening AND statements in parenthesis.

AyanSinhaMahapatraand others added 11 commits June 25, 2025 18:16
@AyanSinhaMahapatra
Add missing wheel builds on release CI
72c7d26
Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
@AyanSinhaMahapatra
Merge pull request#98from aboutcode-org/update-release-ci
4e5b7fd
Add missing wheel builds on release CI
@AyanSinhaMahapatra
Drop python3.9 support and add python 3.14
265e612
Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
@AyanSinhaMahapatra
Support trusted-publishing for package releases
fc4fe3a
Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
@AyanSinhaMahapatra
Update RTD build python version
e81ff6d
Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
@AyanSinhaMahapatra
Merge pull request#99from aboutcode-org/update-python-support
fb75a2e
Drop python3.9 support and add python 3.14
@AyanSinhaMahapatra
Deprecate MacOS-13 CI runners
cf04877
Reference: actions/runner-images#13046
Signed-off-by: Ayan Sinha Mahapatra <asmahapatra@aboutcode.org>
@AyanSinhaMahapatra
Merge pull request#100from aboutcode-org/deprecate-macos-runners
3c2606a
Deprecate MacOS-13 CI runners
@chinyeungli
Typo and spacing corrections.
8682dbf
Signed-off-by: Chin Yeung Li <tli@nexb.com>
@chinyeungli
Merge pull request#102from aboutcode-org/enhance_docstring_clarity
ae254b3
Typo and spacing corrections
@chinyeungli
Fixed licensing dedup issue#67
77bd586
Signed-off-by: Chin Yeung Li <tli@nexb.com>
@chinyeunglichinyeungli mentioned this pull request Mar 16, 2026
Open
chinyeungliand others added 5 commits March 17, 2026 10:42
@chinyeungli
Added more logically equivalent tests#67
ca2a42a
Signed-off-by: Chin Yeung Li <tli@nexb.com>
@AyanSinhaMahapatra
Replace versions with commit hashes in actions
33840a6
Signed-off-by: Ayan Sinha Mahapatra <asmahapatra@aboutcode.org>
@AyanSinhaMahapatra
Merge pull request#103from aboutcode-org/commit-hash-in-ci
fa15c37
Replace tagged versions with commit hashes in github actions
@AyanSinhaMahapatra
Enable zizmor in github actions
c3b24ea
Also add fixes to actions from zizmor runs.
Signed-off-by: Ayan Sinha Mahapatra <asmahapatra@aboutcode.org>
@AyanSinhaMahapatra
Merge pull request#104from aboutcode-org/enable-zizmor
c9cb5a9
Enable zizmor in github actions
This was referenced May 15, 2026
Closed
Open
@AyanSinhaMahapatra
Add tests and simplify comments
819554f
Signed-off-by: Ayan Sinha Mahapatra <asmahapatra@aboutcode.org>
@AyanSinhaMahapatra
Update azure pipelines images and python versions
9f886b9
Signed-off-by: Ayan Sinha Mahapatra <asmahapatra@aboutcode.org>
@AyanSinhaMahapatra
Drop support for python 3.9
abf9ebb
Signed-off-by: Ayan Sinha Mahapatra <asmahapatra@aboutcode.org>
@AyanSinhaMahapatra
Merge remote-tracking branch 'skeleton/main' into 67_dedup_issue
11ce1e2
Signed-off-by: Ayan Sinha Mahapatra <asmahapatra@aboutcode.org>
@github-advanced-security

github-advanced-securityAI commented May 15, 2026

Copy link
Copy Markdown

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

github-advanced-security[bot]
github-advanced-securityBot found potential problems May 15, 2026
View reviewed changes
Comment thread.github/workflows/pypi-release.yml Dismissed
AyanSinhaMahapatra
AyanSinhaMahapatra approved these changes May 15, 2026
View reviewed changes

@AyanSinhaMahapatraAyanSinhaMahapatra left a comment •
edited
Loading

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.

@chinyeungli thanks++ for the fix, this would be quite an update and very useful on the scancode-toolkit side, ref: aboutcode-org/scancode-toolkit#2993

I added some minor updates, and updated tests, CI from skeleton.
Azure pipelines tests were not running for some reason, I've re-enabled them too.
All green, merging!

Comment threadsrc/license_expression/__init__.py
Comment threadtests/test_license_expression.py
@AyanSinhaMahapatra
AyanSinhaMahapatra merged commit 30a9d51 into mainMay 15, 2026
17 of 22 checks passed
@chinyeungli
chinyeungli deleted the 67_dedup_issue branch May 26, 2026 03:13
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AyanSinhaMahapatraAyanSinhaMahapatraAyanSinhaMahapatra approved these changes
@pombredannepombredanneAwaiting requested review from pombredanne
+1 more reviewer
@github-advanced-securitygithub-advanced-security[bot]github-advanced-security[bot] left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@chinyeungli@github-advanced-security@AyanSinhaMahapatra