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

fix: Only fix Windows console with Colorama #1379

Merged
merged 1 commit into from
Oct 29, 2023

Conversation

pawamoy
Copy link
Contributor

@pawamoy pawamoy commented Oct 22, 2023

Closes #1378

Not sure how to test this.

@codecov
Copy link

codecov bot commented Oct 22, 2023

Codecov Report

Merging #1379 (4dff0ce) into master (908c090) will decrease coverage by 0.21%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1379      +/-   ##
==========================================
- Coverage   97.41%   97.20%   -0.21%     
==========================================
  Files          48       48              
  Lines        4327     4327              
==========================================
- Hits         4215     4206       -9     
- Misses        112      121       +9     
Flag Coverage Δ
unittests 97.20% <100.00%> (-0.21%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
copier/tools.py 96.25% <100.00%> (ø)

... and 1 file with indirect coverage changes

Copy link
Member

@yajo yajo left a comment

Choose a reason for hiding this comment

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

I trust you.

@yajo
Copy link
Member

yajo commented Oct 23, 2023

908c090 broke the CI it seems. Gotta fix that.

@yajo yajo merged commit 5cc7f67 into copier-org:master Oct 29, 2023
15 of 18 checks passed
celm1990 added a commit to celm1990/copier that referenced this pull request Nov 11, 2023
After this PR copier-org#1379   method `just_fix_windows_console` is used instead of `init` So is need install `colorama>=0.4.6` because this method is add in commit tartley/colorama@5f1794d
otherwise raise error
AttributeError: module 'colorama' has no attribute 'just_fix_windows_console'
celm1990 added a commit to celm1990/copier that referenced this pull request Nov 11, 2023
After this PR copier-org#1379   method `just_fix_windows_console` is used instead of `init` So is need install `colorama>=0.4.6` because this method is add in commit tartley/colorama@5f1794d
otherwise raise error
AttributeError: module 'colorama' has no attribute 'just_fix_windows_console'
sisp pushed a commit that referenced this pull request Nov 11, 2023
After this PR #1379   method `just_fix_windows_console` is used instead of `init` So is need install `colorama>=0.4.6` because this method is add in commit tartley/colorama@5f1794d
otherwise raise error
AttributeError: module 'colorama' has no attribute 'just_fix_windows_console'
@pawamoy pawamoy deleted the no-colorama-init-import-time branch April 2, 2024 10:42
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.

bug: Importing from Copier prevents forcing colors with FORCE_COLOR=1
2 participants