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

helpers/documentation: don't colorize twice #2182

Merged
merged 1 commit into from Apr 10, 2021
Merged

Conversation

kyrylo
Copy link
Member

@kyrylo kyrylo commented Apr 10, 2021

Fixes #2181 (Bad formatting)

One of the regexps matches against string that were already colored. To prevent
that I used a negative look-ahead and rejected color sequences.

@kyrylo kyrylo force-pushed the 2181-doc-formatting-fix branch 2 times, most recently from 84fa00a to aeb7247 Compare April 10, 2021 22:02
Fixes #2181 (Bad formatting)

One of the regexps matches against string that were already colored. To prevent
that I used a negative look-ahead and rejected color sequences.
@kyrylo kyrylo merged commit 732a930 into master Apr 10, 2021
@kyrylo kyrylo deleted the 2181-doc-formatting-fix branch April 10, 2021 22:03
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.

Bad formatting
1 participant