Skip to content

Conversation

@chinyeungli
Copy link
Contributor

Fixes #4181 - Update ABOUT files

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.

This PR only consist of ABOUT files (i.e. no code changes), I believe the failing tests are as same as the current in the develop branch

  • Commits are in uniquely-named feature branch and has no merge conflicts 📁
  • Updated documentation pages (if applicable)
  • Updated CHANGELOG.rst (if applicable)

Signed-off-by: Chin Yeung Li [email protected]

Signed-off-by: Chin Yeung Li <[email protected]>
…ap.min.css were switched.

 * Updated the ABOUT files to fix the error
 * Added package_url field

Signed-off-by: Chin Yeung Li <[email protected]>
 * Mainly added package_url into ABOUT files
 * Removed "package_url" field if it refers to borrowed code

Signed-off-by: Chin Yeung Li <[email protected]>
Signed-off-by: Chin Yeung Li <[email protected]>
Copy link
Member

@AyanSinhaMahapatra AyanSinhaMahapatra left a comment

Choose a reason for hiding this comment

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

Thanks @chinyeungli LGTM!
We should also have some linter/checker for ABOUT files to check for common issues and missing data, check if the referenced files are present etc. And add this to all repos via skeleton, we should probably open a seperate issue for this.

name: scancode-toolkit
license_expression: apache-2.0 AND cc-by-4.0 AND other-permissive AND other-copyleft
copyright: Copyright (c) nexB Inc. and others.
homepage_url: https://www.aboutcode.org/
Copy link
Member

Choose a reason for hiding this comment

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

Since we are doing a cleanup, we should probably delete this file, since this does not add any data to the package manifests already present here for scancode-toolkit.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

For the purpose of the ABOUT file, I think we still want to keep this file.

@chinyeungli
Copy link
Contributor Author

Thanks @chinyeungli LGTM! We should also have some linter/checker for ABOUT files to check for common issues and missing data, check if the referenced files are present etc. And add this to all repos via skeleton, we should probably open a seperate issue for this.

We can use the about inventory command from the aboutcode-toolkit to look through all the ABOUT files to see if there are any 'custom' fields, along with fields for which we want to have data.

@chinyeungli chinyeungli merged commit e830934 into develop Mar 12, 2025
37 of 40 checks passed
@pombredanne pombredanne deleted the 4181-update_ABOUT_to_use_license_expression branch April 23, 2025 06:25
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.

Update ABOUT files

3 participants