Skip to content
This repository has been archived by the owner on Aug 3, 2023. It is now read-only.

Migrate straggler emojis to terminal::emoji #382

Merged
merged 1 commit into from
Aug 5, 2019

Conversation

EverlastingBugstopper
Copy link
Contributor

@EverlastingBugstopper EverlastingBugstopper commented Aug 2, 2019

This PR changes a few remaining instances where warngler was using hard-coded emojis for messages rather than terminal::emoji. There are two instances where this changes ⛔ to ⚠️, but the rest is simply a change in implementation.

Copy link
Contributor

@gabbifish gabbifish left a comment

Choose a reason for hiding this comment

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

One overarching comment for resolution. I still think we should make error and warn emojis distinct. Otherwise lgtm!

src/commands/publish/route.rs Show resolved Hide resolved
src/commands/publish/route.rs Show resolved Hide resolved
src/commands/subdomain.rs Show resolved Hide resolved
@gabbifish gabbifish self-requested a review August 5, 2019 08:27
Copy link
Contributor

@gabbifish gabbifish left a comment

Choose a reason for hiding this comment

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

LGTM!

@ashleygwilliams ashleygwilliams added this to the 1.1.1 milestone Aug 5, 2019
@EverlastingBugstopper EverlastingBugstopper self-assigned this Aug 5, 2019
@EverlastingBugstopper EverlastingBugstopper merged commit d4be43d into master Aug 5, 2019
@delete-merged-branch delete-merged-branch bot deleted the avery/fix-emojis branch August 5, 2019 22:13
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.

4 participants