Skip to content

Update prog_lexers.py and pygments_lexers.py #4 - #5

Merged
JonoYang merged 10 commits into
mainfrom
4-update-pygments-files
Oct 5, 2020
Merged

Update prog_lexers.py and pygments_lexers.py #4#5
JonoYang merged 10 commits into
mainfrom
4-update-pygments-files

Conversation

@JonoYang

@JonoYang JonoYang commented Oct 2, 2020

Copy link
Copy Markdown
Member
* TODO: Comment out newly added lexers in prog_lexers.py if they are not for programming languages

Signed-off-by: Jono Yang jyang@nexb.com

JonoYang and others added 2 commits October 1, 2020 18:29
    * TODO: Comment out newly added lexers in prog_lexers.py if they are not for programming languages

Signed-off-by: Jono Yang <jyang@nexb.com>
@pombredanne
pombredanne force-pushed the 4-update-pygments-files branch from 8bcc74e to f728d6f Compare October 3, 2020 18:42
pombredanne and others added 8 commits October 3, 2020 20:51
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
* disbale several less interesting Pygments lexers
* mimetype_python is now correctly collected
* is_media_with_meta is now not incorrectly reported


Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Jono Yang <jyang@nexb.com>
Signed-off-by: Jono Yang <jyang@nexb.com>
@JonoYang

JonoYang commented Oct 5, 2020

Copy link
Copy Markdown
Member Author

@pombredanne Thanks for fixing up the test code and adding the test data and expectations! They look good. I also have pinned pygments in setup.cfg and modified configure to install typecode_libmagic after configuring typecode proper.

@JonoYang
JonoYang merged commit 089b568 into main Oct 5, 2020
@JonoYang
JonoYang deleted the 4-update-pygments-files branch October 5, 2020 23:40
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