Skip to content
This repository was archived by the owner on Jan 20, 2025. It is now read-only.

Include normal reset byte in test assertion #51

Merged
merged 1 commit into from
Sep 22, 2020

Conversation

electrofelix
Copy link
Contributor

Test for coloured output will fail due to a mismatch between what is
generated and what is asserted. The generated output will contain a byte
to set console output to normal before subsequently modifying it to be
coloured red at normal intensity. Update the assertion to match.

Test for coloured output will fail due to a mismatch between what is
generated and what is asserted. The generated output will contain a byte
to set console output to normal before subsequently modifying it to be
coloured red at normal intensity. Update the assertion to match.
@electrofelix
Copy link
Contributor Author

Appears this was caused by a PR being merged recently in mgutz/ansi#11, possibly with the change in #50 it would be possible to pin to a specific git sha using go.mod to prevent this in the future.

Copy link
Member

@asottile asottile left a comment

Choose a reason for hiding this comment

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

@asottile asottile merged commit 839c50b into asottile-archive:master Sep 22, 2020
@electrofelix electrofelix deleted the fix-ci branch September 22, 2020 16:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants