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

Update README.md #2306

Merged
merged 1 commit into from
Sep 8, 2022
Merged

Update README.md #2306

merged 1 commit into from
Sep 8, 2022

Conversation

rozenmd
Copy link
Contributor

@rozenmd rozenmd commented Sep 8, 2022

No description provided.

@rozenmd rozenmd requested a review from a team as a code owner September 8, 2022 12:24
@rozenmd rozenmd merged commit f6c1104 into master Sep 8, 2022
@delete-merged-branch delete-merged-branch bot deleted the rozenmd-patch-1 branch September 8, 2022 12:48
@petebacondarwin
Copy link

Drive by comment...
I know that in this case it is a minimal change, but it is really helpful if the commit message has a bit more context.
If in the future, I am looking down the history and see "Update README.md" for this commit, I have to go into the diff to see what actually changed.
Something like "Add README note to point users at the new version of Wrangler" or something would be better.

JacobMGEvans pushed a commit that referenced this pull request Sep 29, 2022
@rozenmd @JacobMGEvans
Update README.md (#2306)
ff6be43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@GregBrimble GregBrimble GregBrimble approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@rozenmd @petebacondarwin @GregBrimble