Skip to content
This repository has been archived by the owner on Aug 31, 2023. It is now read-only.

fix: add missing license to the generated packages on CI #4301

Merged
merged 1 commit into from
Mar 15, 2023
Merged

fix: add missing license to the generated packages on CI #4301

merged 1 commit into from
Mar 15, 2023

Conversation

jorgebodega
Copy link
Contributor

@jorgebodega jorgebodega commented Mar 15, 2023

Summary

This PR just add the license to the autogenerated packages marked as optional dependencies of the main package.

This was causing problems with some systems that require licenses.

This was referenced on this issue.

Closes #4290

Test Plan

AFAIK, this is only executed on CI whenever a new version is deployed, so no tests are required.

Documentation

  • The PR requires documentation
  • I will create a new PR to update the documentation

@netlify
Copy link

netlify bot commented Mar 15, 2023

Deploy Preview for docs-rometools canceled.

Built without sensitive environment variables

Name Link
🔨 Latest commit 2378aec
🔍 Latest deploy log https://app.netlify.com/sites/docs-rometools/deploys/6411e86c29254f00087ff697

@jorgebodega jorgebodega changed the title chore: add missing license to generated packages fix: add missing license to the generated packages on CI Mar 15, 2023
Copy link
Contributor

@ematipico ematipico left a comment

Choose a reason for hiding this comment

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

Thank you!

@ematipico ematipico merged commit 2af0def into rome:main Mar 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🐛 missing license on optional dependencies
2 participants