Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

nturgut
Copy link
Contributor

@nturgut nturgut commented Apr 24, 2020

change the repo fetch script to recognize candidate versions such as flutter-1.17-candidate.3. Originally the script only accepted branches such as v0.7.3 as valid engine branches.

This will fix test breakages such as: #17931

…flutter-1.17-candidate.3. Originally the script only accepted branches such as v0.7.3 as valid engine branches.
@nturgut nturgut requested review from pcsosinski and tvolkert April 24, 2020 22:20
@auto-assign auto-assign bot requested a review from GaryQian April 24, 2020 22:20
@nturgut nturgut removed the request for review from GaryQian April 24, 2020 22:20
Copy link
Contributor

@tvolkert tvolkert left a comment

Choose a reason for hiding this comment

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

LGTM

@nturgut
Copy link
Contributor Author

nturgut commented Apr 25, 2020

Thanks for the review. The tests passed, merging the PR.

@nturgut nturgut merged commit 4f888d6 into flutter:master Apr 25, 2020
pcsosinski pushed a commit to pcsosinski/engine that referenced this pull request Apr 27, 2020
* change the repo fetch script to recognize candidate versions such as flutter-1.17-candidate.3. Originally the script only accepted branches such as v0.7.3 as valid engine branches.

* addressing reviewer comments: changing the release regular expression
pcsosinski pushed a commit to pcsosinski/engine that referenced this pull request May 1, 2020
* change the repo fetch script to recognize candidate versions such as flutter-1.17-candidate.3. Originally the script only accepted branches such as v0.7.3 as valid engine branches.

* addressing reviewer comments: changing the release regular expression
pcsosinski pushed a commit that referenced this pull request May 2, 2020
* use $CIRRUS_BASE_BRANCH for the branch name (#18014)

use $CIRRUS_BASE_BRANCH for getting the branch name when cloning the flutter repo.

* Change the repo fetch script used in integration tests (#17943)

* change the repo fetch script to recognize candidate versions such as flutter-1.17-candidate.3. Originally the script only accepted branches such as v0.7.3 as valid engine branches.

* addressing reviewer comments: changing the release regular expression

Co-authored-by: Nurhan Turgut <nurhan@google.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants