Skip to content

docs(license): add adr for license attribution#3017

Merged
rwaskiewicz merged 2 commits intomasterfrom
rwaskiewicz/license-adr
Aug 24, 2021
Merged

docs(license): add adr for license attribution#3017
rwaskiewicz merged 2 commits intomasterfrom
rwaskiewicz/license-adr

Conversation

@rwaskiewicz
Copy link
Copy Markdown
Member

add an ADR describing how third party projects that are bundled in the
Stencil distributable should be attributed. I confirmed this practice
with Adam as a part of #3016.

@rwaskiewicz rwaskiewicz requested a review from a team August 23, 2021 17:37

## Links

Source: [https://adr.github.io/](https://adr.github.io/)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nix the links section as it's not necessary on this ADR.


## Decision

Third party scripts that are included in the Stencil output shall be placed in the `entryDeps` field in a
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is all great to know, thank you for spelling it out so plainly. Up until now, I had no idea about this.

add an ADR describing how third party projects that are bundled in the
Stencil distributable should be attributed. I confirmed this practice
with Adam as a part of #3016.
@rwaskiewicz rwaskiewicz force-pushed the rwaskiewicz/license-adr branch from 0a10f96 to 0d3311f Compare August 24, 2021 17:16
@rwaskiewicz rwaskiewicz merged commit c3f7f2e into master Aug 24, 2021
@rwaskiewicz rwaskiewicz deleted the rwaskiewicz/license-adr branch August 24, 2021 17:39
rwaskiewicz added a commit that referenced this pull request Aug 27, 2021
add an ADR describing how third party projects that are bundled in the
Stencil distributable should be attributed. I confirmed this practice
with Adam as a part of #3016.
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.

2 participants