Skip to content

fix: yarn library license issues#6483

Merged
d4x1 merged 3 commits into
mainfrom
fix-license-issues
Nov 24, 2023
Merged

fix: yarn library license issues#6483
d4x1 merged 3 commits into
mainfrom
fix-license-issues

Conversation

@abeizn
Copy link
Copy Markdown
Contributor

@abeizn abeizn commented Nov 20, 2023

Summary

fix: yarn library license issues

Does this close any open issues?

Closes na

Screenshots

Include any relevant screenshots here.

Other Information

Any other information that is important to this PR.

pjfanning
pjfanning reviewed Nov 20, 2023
View reviewed changes
Comment thread config-ui/.yarn/releases/yarn-3.4.1.cjs
#!/usr/bin/env node
/* eslint-disable */
//prettier-ignore
/*!

This comment was marked as resolved.

Sign in to view

Copy link
Copy Markdown
Member

@pjfanning pjfanning Nov 21, 2023

Choose a reason for hiding this comment

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

can you at least add a link for the LICENSE file but ideally you should list the licenses here?

Copy link
Copy Markdown
Contributor Author

@abeizn abeizn Nov 22, 2023

Choose a reason for hiding this comment

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

I think so, the bsd license should be listed here just like the apache license is listed in other files

Comment thread LICENSE Outdated
--------------------------------------------------------------------------------

The following file is distributed under the BSD 2-Clause License:
The following file is distributed under the Licenses:

This comment was marked as resolved.

Sign in to view

pjfanning
pjfanning reviewed Nov 21, 2023
View reviewed changes
Comment thread LICENSE
--------------------------------------------------------------------------------

The following file is distributed under the BSD 2-Clause License:
The following file is distributed under different licenses:
Copy link
Copy Markdown
Member

@pjfanning pjfanning Nov 21, 2023

Choose a reason for hiding this comment

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

you should list them all

Copy link
Copy Markdown
Member

@justinmclean justinmclean Nov 21, 2023

Choose a reason for hiding this comment

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

it looks like they have listed them all, although perhaps not quite in the format I would have chosen.

Copy link
Copy Markdown
Contributor Author

@abeizn abeizn Nov 23, 2023

Choose a reason for hiding this comment

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

Hi justin, thank you very much for your question. Regarding the format you mentioned, I would like to ask you to clarify which format you are referring to.

Copy link
Copy Markdown
Contributor Author

@abeizn abeizn Nov 24, 2023

Choose a reason for hiding this comment

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

@justinmclean

@abeizn abeizn added bot/auto-cherry-pick-for-release-v0.19 auto cherry pick pr for release-v0.19 bot/auto-cherry-pick-for-release-v0.20 auto cherry pick pr for release-v0.20 labels Nov 24, 2023
@d4x1 d4x1 merged commit 1986653 into main Nov 24, 2023
@d4x1 d4x1 deleted the fix-license-issues branch November 24, 2023 09:36
@abeizn abeizn added needs-cherrypick-v0.19 and removed bot/auto-cherry-pick-for-release-v0.19 auto cherry pick pr for release-v0.19 bot/auto-cherry-pick-for-release-v0.20 auto cherry pick pr for release-v0.20 labels Nov 24, 2023
github-actions Bot pushed a commit that referenced this pull request Nov 24, 2023
@abeizn
fix: yarn library license issues (#6483)
ddc7150 
* fix: yarn library use license

* fix: typo

* fix: yarn software license issues
@github-actions github-actions Bot mentioned this pull request Nov 24, 2023
Merged
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Nov 24, 2023

🤖 cherry pick finished successfully 🎉!

@github-actions github-actions Bot added the bot/auto-cherry-pick-completed auto cherry pick completed label Nov 24, 2023
abeizn added a commit that referenced this pull request Nov 24, 2023
@github-actions @abeizn
fix: yarn library license issues (#6483) (#6515)
bd548a2 
* fix: yarn library use license

* fix: typo

* fix: yarn software license issues

Co-authored-by: abeizn <zikuan.an@merico.dev>
github-actions Bot pushed a commit that referenced this pull request Nov 24, 2023
@abeizn
fix: yarn library license issues (#6483)
74a6561 
* fix: yarn library use license

* fix: typo

* fix: yarn software license issues
@github-actions github-actions Bot mentioned this pull request Nov 24, 2023
Merged
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Nov 24, 2023

🤖 cherry pick finished successfully 🎉!

@github-actions github-actions Bot added the bot/auto-cherry-pick-completed auto cherry pick completed label Nov 24, 2023
abeizn added a commit that referenced this pull request Nov 24, 2023
@github-actions @abeizn
fix: yarn library license issues (#6483) (#6516)
b314f86 
* fix: yarn library use license

* fix: typo

* fix: yarn software license issues

Co-authored-by: abeizn <zikuan.an@merico.dev>
@xiaomingzhang12 xiaomingzhang12 mentioned this pull request Dec 19, 2023
Closed
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@d4x1 d4x1 d4x1 approved these changes

+2 more reviewers

@justinmclean justinmclean justinmclean left review comments

@pjfanning pjfanning pjfanning left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

bot/auto-cherry-pick-completed auto cherry pick completed needs-cherrypick-v0.20

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@abeizn @justinmclean @d4x1 @pjfanning