Remove outdated/unnecessary files#521
Merged
Merged
Conversation
Now that we have proper documentation, we have no need for this file.
Codecov Report
@@ Coverage Diff @@
## master #521 +/- ##
=======================================
Coverage 35.16% 35.16%
=======================================
Files 210 210
Lines 17840 17840
Branches 2454 2454
=======================================
Hits 6274 6274
Misses 11161 11161
Partials 405 405
Continue to review full report at Codecov.
|
jwiggins
approved these changes
Dec 30, 2020
jwiggins
left a comment
jwiggins
left a comment
Member
There was a problem hiding this comment.
What about any of these files?
enable/pyglet_backend/notes.txt
enable/savage/compliance/w3c_svg_11/notes.txt
enable/savage/readme.txt
docs/kiva/agg/interface.txt
kiva/agg/freetype2/docs/raster.txt
kiva/agg/freetype2/docs/formats.txt
kiva/agg/implementation_notes.txt
kiva/agg/src/todo.txt
kiva/agg/src/swig_questions.txt
kiva/fonttools/tests/data/source.txt
added 8 commits
December 31, 2020 09:46
modified: enable/savage/compliance/README deleted: enable/savage/compliance/w3c_svg_11/notes.txt
and move the information into a ticket on github deleted: enable/savage/readme.txt
the todo file contained a list of tasks to do, tasks done and issues. it is not clear how uptodate the contents of the file actually are so at the moment, we have chosen to remove it from the source tree. deleted: kiva/agg/src/todo.txt
renamed: kiva/agg/implementation_notes.txt -> docs/kiva/agg/implementation_notes.txt
modified: docs/kiva/agg/implementation_notes.txt
Again, we are not sure how uptodate this document is and how relevant the questions (or answers to the questions) are at this moment deleted: kiva/agg/src/swig_questions.txt
modified: MANIFEST.in renamed: kiva/fonttools/tests/data/source.txt -> font_LICENSE.txt renamed: kiva/fonttools/tests/data/fonttools_LICENSE.txt -> fonttools_LICENSE.txt
again, we are not sure how uptodate the document is and how useful any of the information in the document is. deleted: enable/pyglet_backend/notes.txt
rahulporuri
changed the title
Contributor
Author
It looked like useful information so i kept this around
It looks like they're coming from freetype2 source and are not something we added - so i kept them around. I didn't remove them
I moved this into docs/kiva/agg as it looked like useful documentation. |
jwiggins
approved these changes
Dec 31, 2020
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
There are a number of outdated/unnecessary files in the git source which can be removed. This PR removes some (not all) of them.