Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

scancode: version >31.2.4 contains fix for reindexing #15430

Merged
merged 1 commit into from
Jun 1, 2023

Conversation

0xc0170
Copy link
Contributor

@0xc0170 0xc0170 commented Jun 1, 2023

Summary of changes

The workaround is not needed anymore (v32 is breaking this reindexing as they created new command to do that anyway).

Impact of changes

Migration actions required

Documentation


Pull request type

[x] Patch update (Bug fix / Target update / Docs update / Test update / Refactor)
[] Feature update (New feature / Functionality change / New API)
[] Major update (Breaking change E.g. Return code change / API behaviour change)

Test results

[] No Tests required for this change (E.g docs only update)
[x] Covered by existing mbed-os tests (Greentea or Unittest)
[] Tests / results supplied as part of this PR

Reviewers


The workaround is not needed anymore (v32 is breaking this reindexing as they created new command to do that anyway).
@0xc0170
Copy link
Contributor Author

0xc0170 commented Jun 1, 2023

This should fix the recent scancode breakage (appears starting from v32, details can be found https://github.com/nexB/scancode-toolkit/releases/tag/v32.0.0rc1, look for reindex)

@0xc0170 0xc0170 added needs: CI release-type: patch Indentifies a PR as containing just a patch labels Jun 1, 2023
@0xc0170
Copy link
Contributor Author

0xc0170 commented Jun 1, 2023

CI started

@0xc0170 0xc0170 merged commit 9bff970 into master Jun 1, 2023
11 checks passed
@0xc0170 0xc0170 deleted the 0xc0170-patch-scancode branch June 1, 2023 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs: CI release-type: patch Indentifies a PR as containing just a patch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant