Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Reorg LICENSE files to be more discoverable #204

Merged
merged 2 commits into from
Oct 7, 2020
Merged

Reorg LICENSE files to be more discoverable #204

merged 2 commits into from
Oct 7, 2020

Conversation

jeffmcaffer
Copy link
Contributor

Why:

See #203

What's being changed:

Here we move the LICENSE files up to the root and identify the CC-BY-4.0 content license as the "primary" license (e.g., it is in the LICENSE file) since doc is the major role of the repo. Also updated the README and a couple tools that seem to treat license files specially.

Check off the following:

@jeffmcaffer jeffmcaffer requested a review from a team as a code owner October 7, 2020 21:46
@welcome
Copy link

welcome bot commented Oct 7, 2020

Thanks for opening this pull request! A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines.

zeke
zeke approved these changes Oct 7, 2020
View reviewed changes
Copy link
Contributor

@zeke zeke left a comment

Choose a reason for hiding this comment

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

Nice 👍🏼

@zeke zeke merged commit 6da14f2 into main Oct 7, 2020
@zeke zeke deleted the jm/updateLicense branch October 7, 2020 22:11
@github-actions
Copy link
Contributor

github-actions bot commented Oct 7, 2020

Thanks very much for contributing! Your pull request has been merged 🎉 You should see your changes appear on the site in approximately 24 hours. To add your ✨ contribution to the README.md, create a new comment in this PR with:

@all-contributors please add @jeffmcaffer for docs

If you want to, you can use the emoji key to replace docs with a different contribution type.

@allcontributors
Copy link
Contributor

@github-actions[bot]

I could not determine your intention.

Basic usage: @all-contributors please add @Someone for code, doc and infra

For other usages see the documentation

@janiceilene janiceilene mentioned this pull request Oct 7, 2020
Closed
@jeffmcaffer jeffmcaffer mentioned this pull request Oct 7, 2020
Closed
@mwichmann mwichmann mentioned this pull request Oct 8, 2020
Closed
@jeffmcaffer
Copy link
Contributor Author

@all-contributors please add @jeffmcaffer for docs

@allcontributors allcontributors bot mentioned this pull request Oct 8, 2020
Merged
@allcontributors
Copy link
Contributor

@jeffmcaffer

I've put up a pull request to add @jeffmcaffer! 🎉

@mlinksva mlinksva mentioned this pull request Oct 11, 2020
Closed
966566081639
966566081639 reviewed Nov 17, 2020
View reviewed changes
Copy link

@966566081639 966566081639 left a comment

Choose a reason for hiding this comment

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

124 7

@alikkille

This comment was marked as spam.

Sign in to view
This was referenced Jan 21, 2021
Open
Open
@rhsayem075 rhsayem075 mentioned this pull request Jan 26, 2021
Open
This was referenced Jan 27, 2021
Open
Open
Open
@YoYBaBy YoYBaBy mentioned this pull request Feb 3, 2021
Open
@snyk-bot snyk-bot mentioned this pull request Jun 23, 2021
Open
Antonio9212
Antonio9212 reviewed Apr 14, 2022
View reviewed changes
Copy link

@Antonio9212 Antonio9212 left a comment

Choose a reason for hiding this comment

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

LICENSE-CODE

@github github locked as spam and limited conversation to collaborators Apr 15, 2022
@github github unlocked this conversation Apr 15, 2022
@github github locked as spam and limited conversation to collaborators Apr 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Antonio9212 Antonio9212 Antonio9212 left review comments

@966566081639 966566081639 966566081639 left review comments

@zeke zeke zeke approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@jeffmcaffer @alikkille @zeke @966566081639 @RoMeStOsQWE @Antonio9212