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

Add title to the LICENSE file #71

Merged
merged 1 commit into from
Feb 27, 2021
Merged

Add title to the LICENSE file #71

merged 1 commit into from
Feb 27, 2021

Conversation

waldyrious
Copy link
Contributor

The title is not legally mandated, but it's convenient for human consumption, and provides additional metadata; for that reason, it is typically included in the template text of this license, as recommended by OSI, SPDX, choosealicense.com, and others.

Note: This PR is part of my personal project to improve the consistency and visibility of the ISC license in open source projects. See github/choosealicense.com#377 for more details.

@waldyrious
Copy link
Contributor Author

By the way, should I also edit the NOTICE file accordingly?

@jart jart force-pushed the master branch 2 times, most recently from 133d059 to 218ef49 Compare February 27, 2021 20:08
@jart
Copy link
Owner

jart commented Feb 27, 2021

The NOTICE file is now deleted. I forgot to stage the deletion in a recent change.

Thank you for sending this change! I agree with you. Let's merge this.

@jart jart merged commit 98fba07 into jart:master Feb 27, 2021
@waldyrious waldyrious deleted the patch-1 branch February 28, 2021 00:19
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