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.

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

chore: add license #204

Merged
merged 1 commit into from
Feb 10, 2021
Merged

chore: add license #204

merged 1 commit into from
Feb 10, 2021

Conversation

davidlj95
Copy link

What is the purpose of this pull request?

  • Documentation update
  • Bug fix
  • New rule
  • Changes an existing rule
  • Add autofixing to a rule
  • Other, please explain:

Update LICENSE according to license specified in package.json

ISC one (as specified in package.json) from GitHub template
https://github.com/licenses/license-templates/blob/master/templates/isc.txt

What changes did you make? (Give an overview)

With ISC one (as specified in package.json) from GitHub template
https://github.com/licenses/license-templates/blob/master/templates/isc.txt
@@ -0,0 +1,13 @@
Copyright (c) 2020, xjamundx

Choose a reason for hiding this comment

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

Suggested change
Copyright (c) 2020, xjamundx
Copyright (c) 2016, xjamundx

Please review this. I believe it should be 2016, @xjamundx Correct me if I'm wrong and don't know if we should keep your {{github_username}} as the {{ organization }} or it's better to add your name instead i.e., Jamund Ferguson.

Copy link
Contributor

Choose a reason for hiding this comment

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

I think it should be my name, yes, but I can probably do this.

Choose a reason for hiding this comment

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

@xjamundx Awesome, thanks for the update.

@iamdarshshah
Copy link

@davidlj95 Thanks for the PR. 🎉

Let's wait for @xjamundx 's comment on whether the details mentioned in the license are correct or it still needs any change.

@xjamundx xjamundx merged commit 34c2f76 into eslint-community:development Feb 10, 2021
@davidlj95 davidlj95 deleted the add-license branch February 10, 2021 18:23
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.

None yet

3 participants