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

Reduce output from publish command #584

Merged
merged 1 commit into from
Sep 13, 2019
Merged

Reduce output from publish command #584

merged 1 commit into from
Sep 13, 2019

Conversation

EverlastingBugstopper
Copy link
Contributor

Fixes #523

This PR improves the messaging of wrangler publish.

Before:

✨  Built successfully, built project size is 517 bytes.
✨  Successfully published your script.
✨  Success! Your worker was successfully published. You can view it at example.com/*

After:

$ wrangler publish
✨  Built successfully, built project size is 523 bytes.
✨  Successfully published your script to example.com/*

@EverlastingBugstopper EverlastingBugstopper requested a review from a team September 13, 2019 14:27
@EverlastingBugstopper EverlastingBugstopper self-assigned this Sep 13, 2019
xtuc
xtuc approved these changes Sep 13, 2019
View reviewed changes
Copy link
Member

@xtuc xtuc left a comment

Choose a reason for hiding this comment

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

Nice. Also removes a merge conflict 😄

@EverlastingBugstopper EverlastingBugstopper merged commit c8c674b into master Sep 13, 2019
@delete-merged-branch delete-merged-branch bot deleted the avery/sit-down-be-humble branch September 13, 2019 14:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@xtuc xtuc xtuc approved these changes

Assignees

@EverlastingBugstopper EverlastingBugstopper

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

wrangler publish is a bit too excited
2 participants
@EverlastingBugstopper @xtuc