Skip to content

Commit a92db45

Browse files
TechnologicatTechnologicat
committed
remove the local ansicolor module
It was only used internally by the testing framework, and that framework depends on `mcpyrate` anyway (since the `test` constructs are macros), so we can just use `mcpyrate.colorizer`, to DRY out the colorization stuff. This needs an upgrade to `mcpyrate`, because `colorama` 0.4.4 is missing a couple of styles, particularly ITALIC. There's new code in `mcpyrate` master that injects the missing styles (if indeed missing) when the colorizer loads.
1 parent 75f971d commit a92db45

File tree

3 files changed

+106
-177
lines changed

3 files changed

+106
-177
lines changed

‎unpythonic/syntax/__init__.py‎

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -104,10 +104,6 @@
104104

105105
# TODO: grep codebase for "0.15", may have some pending interface changes that don't have their own GitHub issue (e.g. parameter ordering of `unpythonic.it.window`)
106106

107-
# TODO: ansicolor: `mcpyrate` already depends on Colorama anyway (and has a *nix-only fallback capability).
108-
# TODO: `unpythonic` only needs the colorizer in the *macro-enabled* test framework; so we don't really need
109-
# TODO: to provide our own colorizer; we can use the one from `mcpyrate`. (It would be different if regular code needed it.)
110-
111107
# TODO: AST pattern matching for `mcpyrate`? Would make destructuring easier. A writable representation (auto-viewify) is a pain to build, though...
112108

113109
# TODO: 0.16: move `scoped_transform` to `mcpyrate` as `ScopedASTTransformer` and `ScopedASTVisitor`.

‎unpythonic/test/ansicolor.py‎

Lines changed: 0 additions & 95 deletions
This file was deleted.

0 commit comments

Comments
 (0)