-
Notifications
You must be signed in to change notification settings - Fork 229
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
Licence workflow automation #102
Comments
Or is there something simple already in place for this? |
Perhaps it's best to read the license of each direct and transitive dependency and construct a license header containing all of them. Thoughts? |
If it exists in each repo what is the requirement for adding to the style sheet too? |
I'm assuming a workflow like:
This could result in more than 10 different CSS packages being imported, due to transitive dependencies (With the potential of all of them having different licenses), so when the build is done, how do we guarantee that the licenses for each of the individual CSS repositories are reflected in the built file? I think the easiest thing to do, which does not place any additional requirement on the package maintainer, is to read the /*!
normalize.css v3.0.3 | MIT License | github.com/necolas/normalize.css
base.css v1.0.3 | MIT License | github.com/necolas/normalize.css
custom-component.css v1.0.0 | Apache License 2.0 | github.com/bigpimpin/custom-component.css
custom-component-transitive-dependency.css v1.0.0 | GPL | github.com/alig/custom-component-transitive-dependency.css
*/ When the single optimized CSS file is built, the license header would be inserted at the top using something like |
@oleersoy @simonsmith This is a cool idea, but seems unnecessary. The only library dependency that's built into core is normalize.css. If core were to pull in any other dependency, we'd just have to make sure their license allowed it to be distributed under the SUIT CSS license. |
@frekyll I'm assuming a larger eco system of components, utilities, etc. Lets just say that boo publishes a component and foo declares it as an NPM dependency and it becomes part of the css that foo deploys. |
Cross referencing:
suitcss/base#22
The text was updated successfully, but these errors were encountered: