[Docs][UI Next] Add package license.#8545
Conversation
songjianet
added
dependencies
songjianet
changed the title
|
Kudos, SonarCloud Quality Gate passed!
|
|
Should we also add to |
CalvinKirs
left a comment
CalvinKirs
left a comment
There was a problem hiding this comment.
you also need license to dolphinscheduler-dist/release-docs/LICENSE
and add NOTICE to dolphinscheduler-dist/release-docs/NOTICE
In addition, please make sure that the corresponding license corresponds to the version you are referencing
I know I have to deal with this, but I'm working on other things now, I can merge these first, and I'll add them later. |
| @@ -0,0 +1,19 @@ | |||
| Copyright OpenJS Foundation and other contributors, <www.openjsf.org> | |||
There was a problem hiding this comment.
Dependencies in devDependencies don't need to be included in the release licenses
There was a problem hiding this comment.
This piece may be adjusted in the future, so I wrote all the dependencies.
4 participants
Purpose of the pull request
#7332
Brief change log
Verify this pull request
This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows: