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

Cleanup License files #696

Merged
merged 2 commits into from
Apr 20, 2021
Merged

Cleanup License files #696

merged 2 commits into from
Apr 20, 2021

Conversation

rahulporuri
Copy link
Contributor

This PR makes a number of license-file related changes

  • It removes the chaco\LICENSE file completely and moves the matplotlib license into the top level LICENSE.txt file
  • It also moves the content in PngSuite/LICENSE.txt into the new top-level image_LICENSE_PngSuite.txt file.
  • It fixes a few issues with the top-level image_LICENSE.txt
  • It updates the MANIFEST.in file to include the new files.

It also makes a couple of unrelated changes

  • removes enamlc file from the gitignore.
  • fixes a few issues in the README.rst file

Poruri Sai Rahul added 2 commits April 20, 2021 12:58
This commit makes a number of license-file related changes.

It removes the chaco\LICENSE file completely and moves the matplotlib
license into the top level LICENSE.txt file

It also moves the content in PngSuite/LICENSE.txt into the new top-level
image_LICENSE_PngSuite.txt file. Finally, I update the top-level
image_LICENSE.txt file

	modified:   .gitignore
	modified:   LICENSE.txt
	modified:   MANIFEST.in
	modified:   README.rst
	deleted:    chaco/LICENSE
	modified:   image_LICENSE.txt

	deleted:    chaco/tools/toolbars/images/image_LICENSE.txt
	renamed:    chaco/tests/data/PngSuite/LICENSE.txt -> image_LICENSE_PngSuite.txt
	modified:   MANIFEST.in
Copy link
Contributor

@aaronayres35 aaronayres35 left a comment

Choose a reason for hiding this comment

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

LGTM

@rahulporuri rahulporuri merged commit 54ecbfa into master Apr 20, 2021
@rahulporuri rahulporuri deleted the cln/license-updates branch April 20, 2021 15:01
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