From 1dd2374c1307667448ac7b0e79fb80dc6fa9bbd7 Mon Sep 17 00:00:00 2001 From: Chris Yang Date: Thu, 8 Apr 2021 14:27:38 -0700 Subject: [PATCH 1/9] Update PULL_REQUEST_TEMPLATE.md Fixes https://github.com/flutter/flutter/issues/80081 --- .github/PULL_REQUEST_TEMPLATE.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md index 84b9fd214751..0af6f9ba2c08 100644 --- a/.github/PULL_REQUEST_TEMPLATE.md +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -8,7 +8,7 @@ - [ ] I read the [Contributor Guide] and followed the process outlined there for submitting PRs. - [ ] I read the [Tree Hygiene] wiki page, which explains my responsibilities. -- [ ] I read and followed the [Flutter Style Guide] and the [C++, Objective-C, Java style guides]. +- [ ] I read and followed the [Flutter Style Guide] and the [C++, Objective-C, Java style guides]. (Note that not like the flutter/flutter repo, we do use dart formatter in the flutter/plugins repo.) - [ ] I signed the [CLA]. - [ ] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. `[shared_preferences]` - [ ] I listed at least one issue that this PR fixes in the description above. From 802d1fec1e06a7a45a6e03275723587e8599bbac Mon Sep 17 00:00:00 2001 From: Chris Yang Date: Thu, 8 Apr 2021 15:04:39 -0700 Subject: [PATCH 2/9] Update PULL_REQUEST_TEMPLATE.md --- .github/PULL_REQUEST_TEMPLATE.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md index 0af6f9ba2c08..1ffc25ee7be0 100644 --- a/.github/PULL_REQUEST_TEMPLATE.md +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -8,7 +8,7 @@ - [ ] I read the [Contributor Guide] and followed the process outlined there for submitting PRs. - [ ] I read the [Tree Hygiene] wiki page, which explains my responsibilities. -- [ ] I read and followed the [Flutter Style Guide] and the [C++, Objective-C, Java style guides]. (Note that not like the flutter/flutter repo, we do use dart formatter in the flutter/plugins repo.) +- [ ] I read and followed the [Flutter Style Guide] and the [C++, Objective-C, Java style guides]. (Note that not like the flutter/flutter repo, we do use dart formatter in the flutter/plugins repo. Run `dart run ./script/tool/lib/src/main.dart format` under plugins/ to auto format all source code.) - [ ] I signed the [CLA]. - [ ] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. `[shared_preferences]` - [ ] I listed at least one issue that this PR fixes in the description above. From 2c009a5341f3157c5d04e399db9fed217609fe90 Mon Sep 17 00:00:00 2001 From: Chris Yang Date: Thu, 8 Apr 2021 15:10:57 -0700 Subject: [PATCH 3/9] Update PULL_REQUEST_TEMPLATE.md --- .github/PULL_REQUEST_TEMPLATE.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md index 1ffc25ee7be0..45725e762303 100644 --- a/.github/PULL_REQUEST_TEMPLATE.md +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -8,7 +8,7 @@ - [ ] I read the [Contributor Guide] and followed the process outlined there for submitting PRs. - [ ] I read the [Tree Hygiene] wiki page, which explains my responsibilities. -- [ ] I read and followed the [Flutter Style Guide] and the [C++, Objective-C, Java style guides]. (Note that not like the flutter/flutter repo, we do use dart formatter in the flutter/plugins repo. Run `dart run ./script/tool/lib/src/main.dart format` under plugins/ to auto format all source code.) +- [ ] I read and followed the [Flutter Style Guide] and the [C++, Objective-C, Java style guides]. (Note that unlike the flutter/flutter repo, we do use dart formatter in the flutter/plugins repo. Run `dart run ./script/tool/lib/src/main.dart format` under plugins/ to auto format all source code.) - [ ] I signed the [CLA]. - [ ] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. `[shared_preferences]` - [ ] I listed at least one issue that this PR fixes in the description above. From 1569e8a135ede26cd30c99873f02f05a8a06bb27 Mon Sep 17 00:00:00 2001 From: Chris Yang Date: Wed, 14 Apr 2021 11:11:05 -0700 Subject: [PATCH 4/9] review --- .github/PULL_REQUEST_TEMPLATE.md | 2 +- CONTRIBUTING.md | 61 ++++++++------------------------ script/tool/README.md | 52 +++++++++++++++++++++++++++ 3 files changed, 68 insertions(+), 47 deletions(-) diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md index 45725e762303..2fd106c978e6 100644 --- a/.github/PULL_REQUEST_TEMPLATE.md +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -8,7 +8,7 @@ - [ ] I read the [Contributor Guide] and followed the process outlined there for submitting PRs. - [ ] I read the [Tree Hygiene] wiki page, which explains my responsibilities. -- [ ] I read and followed the [Flutter Style Guide] and the [C++, Objective-C, Java style guides]. (Note that unlike the flutter/flutter repo, we do use dart formatter in the flutter/plugins repo. Run `dart run ./script/tool/lib/src/main.dart format` under plugins/ to auto format all source code.) +- [ ] I read and followed the [Flutter Style Guide] and the [C++, Objective-C, Java style guides]. (Note that unlike the flutter/flutter repo, we do use dart formatter in the flutter/plugins repo. See [plugin_tool format](../script/tool/README.md#format-code)) - [ ] I signed the [CLA]. - [ ] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. `[shared_preferences]` - [ ] I listed at least one issue that this PR fixes in the description above. diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index f307695b5956..d0568bc53935 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -48,6 +48,14 @@ for any of the following plugins, we encourage you to submit it fetch from the master repository, not your clone, when running `git fetch` et al.) + +## Setting up toolings + + +We use a range of tooling script to do varies of things on CI. (testing, formatting, etc.) +Your are likely to use some of the tooling script locally in your contributing journey. +See [plugin_tools](./script/tool/README.MD) for more details + ## Running the examples @@ -137,13 +145,7 @@ pod lib lint --allow-warnings XCUITests aren't usually configured with cocoapods in this repo. They are configured in a xcode workspace target named RunnerUITests. To run all the XCUITests in a plugin, follow the steps in a regular iOS development workflow [here](https://developer.apple.com/library/archive/documentation/DeveloperTools/Conceptual/testing_with_xcode/chapters/05-running_tests.html) -For convenience, a [flutter_plugin_tools](https://pub.dev/packages/flutter_plugin_tools) command `xctest` could also be used to run all the XCUITests in the repo: - -```console -pub global activate flutter_plugin_tools -cd /packages -pub global run flutter_plugin_tools xctest --target RunnerUITests --skip -``` +For convenience, a [plugin_tools](./script/tool/README.MD) command [xctest](./script/tool/README.MD#run-xctests) could also be used to run all the XCUITests in the repo. ## Contributing code @@ -159,12 +161,12 @@ To start working on a patch: * `git fetch upstream` * `git checkout upstream/master -b ` * Hack away. - * Verify changes with [flutter_plugin_tools](https://pub.dev/packages/flutter_plugin_tools) + * Verify changes with [plugin_tools](./script/tool/README.MD). ``` -pub global activate flutter_plugin_tools -pub global run flutter_plugin_tools format --plugins plugin_name -pub global run flutter_plugin_tools analyze --plugins plugin_name -pub global run flutter_plugin_tools test --plugins plugin_name +$ cd script/tool & pub get +$ dart run ./script/tool/lib/src/main.dart format --plugins plugin_name +$ pub run ./script/tool/lib/src/main.dart analyze --plugins plugin_name +$ pub run ./script/tool/lib/src/main.dart test --plugins plugin_name ``` * `git commit -a -m ""` * `git push origin ` @@ -245,37 +247,4 @@ Releasing a package is a two-step process. done manually with `git tag $tagname && git push upstream $tagname` while checked out on the commit that updated `version` in `pubspec.yaml`. -We've recently updated -[flutter_plugin_tools](https://github.com/flutter/plugin_tools) to wrap both of -those steps into one command to make it a little easier. This new tool is -experimental. Feel free to fall back on manually running `pub publish` and -creating and pushing the tag in git if there are issues with it. - -Install the tool by running: - -```terminal -$ pub global activate flutter_plugin_tools -``` - -Then, from the root of your local `flutter/plugins` repo, use the tool to -publish a release. - -```terminal -$ pub global run flutter_plugin_tools publish-plugin --package $package -``` - -By default the tool tries to push tags to the `upstream` remote, but that and -some additional settings can be configured. Run `pub global activate -flutter_plugin_tools --help` for more usage information. - -The tool wraps `pub publish` for pushing the package to pub, and then will -automatically use git to try and create and push tags. It has some additional -safety checking around `pub publish` too. By default `pub publish` publishes -_everything_, including untracked or uncommitted files in version control. -`flutter_plugin_tools publish-plugin` will first check the status of the local -directory and refuse to publish if there are any mismatched files with version -control present. - -There is a lot about this process that is still to be desired. Some top level -items are being tracked in -[flutter/flutter#27258](https://github.com/flutter/flutter/issues/27258). +For convenience, a [publish-plugin](./script/tool/README.MD#publish-and-tag-release) tool script could be used to do both step at once. diff --git a/script/tool/README.md b/script/tool/README.md index 162ca0d98a74..a21099f0cc62 100644 --- a/script/tool/README.md +++ b/script/tool/README.md @@ -1,8 +1,60 @@ # Flutter Plugin Tools +Note: The commands in tools are designed to run under plugins/ or plugins/packages/. + To run the tool: ```sh dart pub get dart run lib/src/main.dart ``` + +## Format Code + +```sh +cd /packages +dart run ./script/tool/lib/src/main.dart format --plugins plugin_name +``` + +## Run static analyzer + +```sh +cd /packages +pub run ./script/tool/lib/src/main.dart analyze --plugins plugin_name +``` + +## Run dart unit tests + +```sh +cd /packages +pub run ./script/tool/lib/src/main.dart test --plugins plugin_name +``` + +## Run XCTests + +```sh +cd /packages +dart run lib/src/main.dart xctest --target RunnerUITests --skip +``` + +## Publish and tag release + +``sh +cd /packages +dart run ./script/tool/lib/src/main.dart publish-plugin --package $package +`` + +By default the tool tries to push tags to the `upstream` remote, but that and +some additional settings can be configured. Run `dart run ./script/tool/lib/src/main.dart publish-plugin --help` for more usage information. + +The tool wraps `pub publish` for pushing the package to pub, and then will +automatically use git to try and create and push tags. It has some additional +safety checking around `pub publish` too. By default `pub publish` publishes +_everything_, including untracked or uncommitted files in version control. +`publish-plugin` will first check the status of the local +directory and refuse to publish if there are any mismatched files with version +control present. + +There is a lot about this process that is still to be desired. Some top level +items are being tracked in +[flutter/flutter#27258](https://github.com/flutter/flutter/issues/27258). \ No newline at end of file From 5ed212462d67873261bf9a85112bb85837a8da1e Mon Sep 17 00:00:00 2001 From: Chris Yang Date: Wed, 14 Apr 2021 11:12:40 -0700 Subject: [PATCH 5/9] fix typo --- CONTRIBUTING.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index d0568bc53935..712563169b15 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -49,7 +49,7 @@ for any of the following plugins, we encourage you to submit it et al.) -## Setting up toolings +## Setting up tools We use a range of tooling script to do varies of things on CI. (testing, formatting, etc.) From c815fa2a59fac08e7041defa1178f77d71054a66 Mon Sep 17 00:00:00 2001 From: Chris Yang Date: Thu, 15 Apr 2021 12:48:43 -0700 Subject: [PATCH 6/9] review --- CONTRIBUTING.md | 17 ++++++----------- script/tool/README.md | 8 ++++---- 2 files changed, 10 insertions(+), 15 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 712563169b15..15ce4b6d832b 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -1,6 +1,5 @@ # Contributing to Flutter Plugins - [![Build Status](https://api.cirrus-ci.com/github/flutter/plugins.svg)](https://cirrus-ci.com/github/flutter/plugins/master) _See also: [Flutter's code of conduct](https://github.com/flutter/flutter/blob/master/CODE_OF_CONDUCT.md)_ @@ -26,14 +25,12 @@ for any of the following plugins, we encourage you to submit it ## Things you will need - * Linux, Mac OS X, or Windows. * git (used for source version control). * An ssh client (used to authenticate with GitHub). ## Getting the code and configuring your environment - * Ensure all the dependencies described in the previous section are installed. * Fork `https://github.com/flutter/plugins` into your own GitHub account. If you already have a fork, and are now installing a development environment on @@ -51,14 +48,12 @@ for any of the following plugins, we encourage you to submit it ## Setting up tools - We use a range of tooling script to do varies of things on CI. (testing, formatting, etc.) Your are likely to use some of the tooling script locally in your contributing journey. -See [plugin_tools](./script/tool/README.MD) for more details +See [plugin_tools](./script/tool/README.MD) for more details. ## Running the examples - To run an example with a prebuilt binary from the cloud, switch to that example's directory, run `pub get` to make sure its dependencies have been downloaded, and use `flutter run`. Make sure you have a device connected over @@ -162,11 +157,11 @@ To start working on a patch: * `git checkout upstream/master -b ` * Hack away. * Verify changes with [plugin_tools](./script/tool/README.MD). -``` -$ cd script/tool & pub get -$ dart run ./script/tool/lib/src/main.dart format --plugins plugin_name -$ pub run ./script/tool/lib/src/main.dart analyze --plugins plugin_name -$ pub run ./script/tool/lib/src/main.dart test --plugins plugin_name +```sh +cd script/tool & pub get +dart run ./script/tool/lib/src/main.dart format --plugins plugin_name +pub run ./script/tool/lib/src/main.dart analyze --plugins plugin_name +pub run ./script/tool/lib/src/main.dart test --plugins plugin_name ``` * `git commit -a -m ""` * `git push origin ` diff --git a/script/tool/README.md b/script/tool/README.md index a21099f0cc62..03dbd2264d07 100644 --- a/script/tool/README.md +++ b/script/tool/README.md @@ -41,11 +41,11 @@ dart run lib/src/main.dart xctest --target RunnerUITests --skip /packages -dart run ./script/tool/lib/src/main.dart publish-plugin --package $package +dart run ./script/tool/lib/src/main.dart publish-plugin --package `` -By default the tool tries to push tags to the `upstream` remote, but that and -some additional settings can be configured. Run `dart run ./script/tool/lib/src/main.dart publish-plugin --help` for more usage information. +By default the tool tries to push tags to the `upstream` remote, but some +additional settings can be configured. Run `dart run ./script/tool/lib/src/main.dart publish-plugin --help` for more usage information. The tool wraps `pub publish` for pushing the package to pub, and then will automatically use git to try and create and push tags. It has some additional @@ -57,4 +57,4 @@ control present. There is a lot about this process that is still to be desired. Some top level items are being tracked in -[flutter/flutter#27258](https://github.com/flutter/flutter/issues/27258). \ No newline at end of file +[flutter/flutter#27258](https://github.com/flutter/flutter/issues/27258). From 2c645ea741a921f201e1a497f7356871b9c9510c Mon Sep 17 00:00:00 2001 From: Chris Yang Date: Thu, 15 Apr 2021 12:53:26 -0700 Subject: [PATCH 7/9] fix --- CONTRIBUTING.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 15ce4b6d832b..f5fa0abb2dd8 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -158,7 +158,7 @@ To start working on a patch: * Hack away. * Verify changes with [plugin_tools](./script/tool/README.MD). ```sh -cd script/tool & pub get +cd script/tool && pub get dart run ./script/tool/lib/src/main.dart format --plugins plugin_name pub run ./script/tool/lib/src/main.dart analyze --plugins plugin_name pub run ./script/tool/lib/src/main.dart test --plugins plugin_name From f7fe93550902ce9e15ad1c66e60cfd7ffade1104 Mon Sep 17 00:00:00 2001 From: Chris Yang Date: Mon, 19 Apr 2021 10:01:41 -0700 Subject: [PATCH 8/9] review --- .github/PULL_REQUEST_TEMPLATE.md | 2 +- CONTRIBUTING.md | 18 ++++++++---------- script/tool/README.md | 21 +++++++++++---------- 3 files changed, 20 insertions(+), 21 deletions(-) diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md index 2fd106c978e6..ec9a6dd6ab92 100644 --- a/.github/PULL_REQUEST_TEMPLATE.md +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -8,7 +8,7 @@ - [ ] I read the [Contributor Guide] and followed the process outlined there for submitting PRs. - [ ] I read the [Tree Hygiene] wiki page, which explains my responsibilities. -- [ ] I read and followed the [Flutter Style Guide] and the [C++, Objective-C, Java style guides]. (Note that unlike the flutter/flutter repo, we do use dart formatter in the flutter/plugins repo. See [plugin_tool format](../script/tool/README.md#format-code)) +- [ ] I read and followed the [Flutter Style Guide] and the [C++, Objective-C, Java style guides]. (Note that unlike the flutter/flutter repo, the flutter/plugins repo does use `dart format`. See [plugin_tool format](../script/tool/README.md#format-code)) - [ ] I signed the [CLA]. - [ ] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. `[shared_preferences]` - [ ] I listed at least one issue that this PR fixes in the description above. diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index f5fa0abb2dd8..ab667ce3a8b2 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -49,7 +49,7 @@ for any of the following plugins, we encourage you to submit it ## Setting up tools We use a range of tooling script to do varies of things on CI. (testing, formatting, etc.) -Your are likely to use some of the tooling script locally in your contributing journey. +There are scripts for many common tasks (testing, formatting, etc.) that will likely be useful in preparing a PR. See [plugin_tools](./script/tool/README.MD) for more details. ## Running the examples @@ -158,10 +158,10 @@ To start working on a patch: * Hack away. * Verify changes with [plugin_tools](./script/tool/README.MD). ```sh -cd script/tool && pub get -dart run ./script/tool/lib/src/main.dart format --plugins plugin_name -pub run ./script/tool/lib/src/main.dart analyze --plugins plugin_name -pub run ./script/tool/lib/src/main.dart test --plugins plugin_name +cd script/tool && pub get && cd ../../ +dart ./script/tool/lib/src/main.dart format --plugins plugin_name +dart ./script/tool/lib/src/main.dart analyze --plugins plugin_name +dart ./script/tool/lib/src/main.dart test --plugins plugin_name ``` * `git commit -a -m ""` * `git push origin ` @@ -191,8 +191,6 @@ You may also add it to the AUTHORS file for [the repository](AUTHORS). ### The review process -* This is a new process we are currently experimenting with, feedback on the process is welcomed at the Gitter contributors channel. * - Reviewing PRs often requires a non trivial amount of time. We prioritize issues, not PRs, so that we use our maintainers' time in the most impactful way. Issues pertaining to this repository are managed in the [flutter/flutter issue tracker and are labeled with "plugin"](https://github.com/flutter/flutter/issues?q=is%3Aopen+is%3Aissue+label%3Aplugin+sort%3Areactions-%2B1-desc). Non trivial PRs should have an associated issue that will be used for prioritization. See the [prioritization section](https://github.com/flutter/flutter/wiki/Issue-hygiene#prioritization) in the Flutter wiki to understand how issues are prioritized. Newly opened PRs first go through initial triage which results in one of: @@ -234,12 +232,12 @@ Some things to keep in mind before publishing the release: from people that immediately adopt it, and uncovering and resolving those support issues will take more time if you're unavailable. -Releasing a package is a two-step process. +To release a package, a [publish-plugin](./script/tool/README.MD#publish-and-tag-release) tool script should be used. This command publishes the new version to pub.dev, and tags the commit in the format of `-v` then pushes it to upstream. + +Alternatively, one can release a package in the below 2-step process. 1. Push the package update to [pub.dev](https://pub.dev) using `pub publish`. 2. Tag the commit with git in the format of `-v`, and then push the tag to the `flutter/plugins` master branch. This can be done manually with `git tag $tagname && git push upstream $tagname` while checked out on the commit that updated `version` in `pubspec.yaml`. - -For convenience, a [publish-plugin](./script/tool/README.MD#publish-and-tag-release) tool script could be used to do both step at once. diff --git a/script/tool/README.md b/script/tool/README.md index 03dbd2264d07..c9491f7401fb 100644 --- a/script/tool/README.md +++ b/script/tool/README.md @@ -1,46 +1,47 @@ # Flutter Plugin Tools -Note: The commands in tools are designed to run under plugins/ or plugins/packages/. +Note: The commands in tools are designed to run at the root of the repository or `/packages/`. To run the tool: ```sh -dart pub get -dart run lib/src/main.dart +cd /script/tool && dart pub get && cd ../../ +dart run ./script/tool/lib/src/main.dart ``` ## Format Code ```sh -cd /packages -dart run ./script/tool/lib/src/main.dart format --plugins plugin_name +cd +dart run /script/tool/lib/src/main.dart format --plugins plugin_name ``` ## Run static analyzer ```sh -cd /packages +cd pub run ./script/tool/lib/src/main.dart analyze --plugins plugin_name ``` -## Run dart unit tests +## Run Dart unit tests ```sh -cd /packages +cd pub run ./script/tool/lib/src/main.dart test --plugins plugin_name ``` ## Run XCTests ```sh -cd /packages +cd dart run lib/src/main.dart xctest --target RunnerUITests --skip ``` ## Publish and tag release ``sh -cd /packages +cd +git checkout dart run ./script/tool/lib/src/main.dart publish-plugin --package `` From 8408aa325a2c1a4f2b3796a6d1e6670a4de96519 Mon Sep 17 00:00:00 2001 From: Chris Yang Date: Mon, 19 Apr 2021 10:22:45 -0700 Subject: [PATCH 9/9] review 2 --- CONTRIBUTING.md | 1 - script/tool/README.md | 4 ++-- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index ab667ce3a8b2..9524af1c2f31 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -48,7 +48,6 @@ for any of the following plugins, we encourage you to submit it ## Setting up tools -We use a range of tooling script to do varies of things on CI. (testing, formatting, etc.) There are scripts for many common tasks (testing, formatting, etc.) that will likely be useful in preparing a PR. See [plugin_tools](./script/tool/README.MD) for more details. diff --git a/script/tool/README.md b/script/tool/README.md index c9491f7401fb..daea4d86390b 100644 --- a/script/tool/README.md +++ b/script/tool/README.md @@ -41,7 +41,7 @@ dart run lib/src/main.dart xctest --target RunnerUITests --skip -git checkout +git checkout dart run ./script/tool/lib/src/main.dart publish-plugin --package `` @@ -49,7 +49,7 @@ By default the tool tries to push tags to the `upstream` remote, but some additional settings can be configured. Run `dart run ./script/tool/lib/src/main.dart publish-plugin --help` for more usage information. The tool wraps `pub publish` for pushing the package to pub, and then will -automatically use git to try and create and push tags. It has some additional +automatically use git to try to create and push tags. It has some additional safety checking around `pub publish` too. By default `pub publish` publishes _everything_, including untracked or uncommitted files in version control. `publish-plugin` will first check the status of the local