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

Migrate NOTICE.md to License report #32

Merged
merged 20 commits into from
Oct 25, 2024

Conversation

masutaka
Copy link
Member

@masutaka masutaka commented Oct 21, 2024

Summary

Up until now, I used to manually put licenses that might need a credit notation in Notice.md.

I added the license report docs/packages-license.md and made it auto-update so that Notice.md is no longer needed.

Also I stopped uploading license reports as artifact.

Related Issue

Nothing

Testing

docs/packages-license.md was successfully committed by GitHub Actions.

https://github.com/route06inc/giselle/blob/migrate-notice-md-to-license-html-report/docs/packages-license.md

Other Information

  • HTML files are not rendered on GitHub, so I committed a Markdown file
  • There was a problem that committing in GitHub Actions did not trigger required checks that should be triggered by that commit. Therefore, I created a GitHub App to create a token for each commit
    • Registered the Repository variable CI_TRIGGER_APP_ID
    • Registered Repository secret CI_TRIGGER_APP_PRIVATE_KEY
  • Changed dorny/paths-filter to actions/cache to avoid infinite execution of GitHub Actions

@masutaka masutaka self-assigned this Oct 21, 2024
Copy link

vercel bot commented Oct 21, 2024

@masutaka is attempting to deploy a commit to the Edge team on Vercel, but is not a member of this team. To resolve this issue, you can:

  • Make your repository public. Collaboration is free for open source and public repositories.
  • Add @masutaka as a member. A Pro subscription is required to access Vercel's collaborative features.
    • If you're the owner of the team, click here and add @masutaka as a member.
    • If you're the user who initiated this build request, click here to request access.
    • If you're already a member of the Edge team, make sure that your Vercel account is connected to your GitHub account.

To read more about collaboration on Vercel, click here.

@masutaka masutaka force-pushed the migrate-notice-md-to-license-html-report branch 5 times, most recently from 6962983 to 2a1b57a Compare October 21, 2024 11:10
@masutaka masutaka force-pushed the migrate-notice-md-to-license-html-report branch from 2a1b57a to 42155d0 Compare October 22, 2024 06:42
Copy link

vercel bot commented Oct 22, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
giselle ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 24, 2024 10:33am

@masutaka masutaka force-pushed the migrate-notice-md-to-license-html-report branch from ebf0b84 to 43621b2 Compare October 22, 2024 07:01
@masutaka masutaka changed the title Migrate NOTICE.md to License html report Migrate NOTICE.md to License markdown report Oct 22, 2024
license_finder permitted_licenses add 'LGPL-3.0-or-later' --why 'To not conflict with Apache-2.0 license. https://opensource.org/license/LGPL-3.0'
$ license_finder permitted_licenses add 'CC0 1.0 Universal' --why 'To not conflict with Apache-2.0 license. https://creativecommons.org/publicdomain/zero/1.0/'
$ license_finder permitted_licenses add 'CC-BY-4.0' --why 'To not conflict with Apache-2.0 license. https://creativecommons.org/licenses/by/4.0/'
@masutaka masutaka changed the title Migrate NOTICE.md to License markdown report Migrate NOTICE.md to License report Oct 22, 2024
$ docker run --rm -v $(pwd):/repo --workdir /repo rhysd/actionlint:latest
.github/workflows/license.yml:75:211: "github.event.pull_request.head.ref" is potentially untrusted. avoid using it
directly in inline scripts. instead, pass it through an environment variable.
see https://docs.github.com/en/actions/security-guides/security-hardening-for-github-actions for more details [expression]
@masutaka masutaka marked this pull request as ready for review October 24, 2024 10:33
Copy link
Member

@shige shige left a comment

Choose a reason for hiding this comment

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

LGTM 🆙

Copy link
Contributor

@toyamarinyon toyamarinyon left a comment

Choose a reason for hiding this comment

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

Great!!!

@masutaka
Copy link
Member Author

Thanks for your review!

@masutaka masutaka merged commit 14d51ae into main Oct 25, 2024
9 checks passed
@masutaka masutaka deleted the migrate-notice-md-to-license-html-report branch October 25, 2024 01:22
@masutaka
Copy link
Member Author

✅ Testing on the main branch was as expected; docs/packages-license.md was not updated.
https://github.com/route06inc/giselle/actions/runs/11510164791/job/32041505556

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.

3 participants