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

cleanup #1102

Merged
merged 2 commits into from
Jan 23, 2019
Merged

cleanup #1102

merged 2 commits into from
Jan 23, 2019

Conversation

cyanglaz
Copy link
Contributor

No description provided.

@cbracken
Copy link
Member

lgtm

@cyanglaz cyanglaz changed the title clenaup cleanup Jan 23, 2019
@cyanglaz cyanglaz merged commit 13c87f8 into flutter:master Jan 23, 2019
@cyanglaz cyanglaz deleted the url_launcher_cleanup branch January 23, 2019 01:53
@@ -1,3 +1,7 @@
## 4.1.0+1

* Clean up.
Copy link
Contributor

Choose a reason for hiding this comment

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

Not sure I'll know what to expect when reading this in a change log, maybe worth being a little more explicit here.

And somewhat related - I think that it's generally helpful when the PR description / commit message say what was done, in this case I think "Bump url_lanuncher's version" would have been a good description (specifically I pulled up this PR as it triggered a build failure, with a more detailed description I could probably know its not this PR fault just from the message)

andreidiaconu pushed a commit to andreidiaconu/plugins that referenced this pull request Feb 17, 2019
andreidiaconu added a commit to andreidiaconu/plugins that referenced this pull request Feb 17, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
4 participants