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: Upgrade github.com/briandowns/spinner to v1.17.0 #8

Merged

Conversation

strider
Copy link
Collaborator

@strider strider commented Jan 21, 2022

In commit [1], we've had to downgrade that package due to a rendering
bug[2]. This latter has been fixed upstream[3] and released in the v1.17.0[4].
We can now 'revert' the downgrade to v1.13.0 by upgrading directly to
the v1.17.0.

[1] 2ed9d23
[2] briandowns/spinner#123
[3] briandowns/spinner#126
[4] https://github.com/briandowns/spinner/releases/tag/v1.17.0

Test done with the v1.17.0:
Screen Shot 2022-01-21 at 2 21 42 PM

In commit [1], we've had to downgrade that package due to a rendering
bug[2]. This latter has been fixed upstream[3] and released in the v1.17.0[4].
We can now 'revert' the downgrade to v1.13.0 by upgrading directly to
the v1.17.0.

[1] 2fc8d7d
[2] briandowns/spinner#123
[3] briandowns/spinner#126
[4] https://github.com/briandowns/spinner/releases/tag/v1.17.0

Signed-off-by: Gael Chamoulaud (Strider) <gchamoul@redhat.com>
@strider strider requested a review from subpop January 21, 2022 15:00
@subpop subpop merged commit 5ff0cd6 into RedHatInsights:main Jan 21, 2022
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.

None yet

2 participants