Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add missing concrete implementation of abstract class #14

Open
wants to merge 45 commits into
base: master
Choose a base branch
from

Conversation

ryanawhelan
Copy link

add missing concrete implementation of abstract class

List which issues are fixed by this PR. You must list at least one issue.

If you had to change anything in the flutter/tests repo, include a link to the migration guide as per the breaking change policy.

Pre-launch Checklist

  • 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, including Features we expect every widget to implement.
  • I signed the CLA.
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or this PR is test-exempt.
  • All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

godofredoc and others added 30 commits November 12, 2021 11:50
…lutter#93448)

* Added SharedAppData to the widgets library (flutter#93175)

* 'add branch flutter-2.8-candidate.3 to enabled_branches in .ci.yaml'

* 'Update Engine revision to 09f1520 for beta release 2.8.0-3.1.pre'

* Pin to specific plugin version in multidex test (flutter#93148)

Co-authored-by: Hans Muller <[email protected]>
Co-authored-by: Emmanuel Garcia <[email protected]>
…lutter#93841)

* [flutter_tools] Catch lack of flutter tools source missing (flutter#93168)

* 'Update Engine revision to bcc2b7f for beta release 2.8.0-3.2.pre'

* Replace text directionality control characters with escape sequences in the semantics_tester (flutter#93034)

Co-authored-by: Christopher Fujino <[email protected]>
Co-authored-by: Jason Simmons <[email protected]>
…lutter#94490)

* make CIPD url customizable using FLUTTER_STORAGE_BASE_URL (flutter#94137)

* 'Update Engine revision to 06a7363 for beta release 2.8.0-3.3.pre'

* Update plugin lint test for federated url_launcher plugin (flutter#94374)

Co-authored-by: Yegor <[email protected]>
Co-authored-by: Jenn Magder <[email protected]>
* Update configurations to match the current test beds.

* Update cirrus secrets.

* Disable mac_module_test_ios. The fix for the test not available in this
branch.
…r#95375)

* Build Flutter iOS plugins with all valid architectures (flutter#95293)

* 'add branch flutter-2.8-candidate.3 to enabled_branches in .ci.yaml'

* 'Update Engine revision to 890a5fca2e34db413be624fc83aeea8e61d42ce6 for stable release 2.8.1'

Co-authored-by: Jenn Magder <[email protected]>
…lutter#93448)

* Added SharedAppData to the widgets library (flutter#93175)

* 'add branch flutter-2.8-candidate.3 to enabled_branches in .ci.yaml'

* 'Update Engine revision to 09f1520 for beta release 2.8.0-3.1.pre'

* Pin to specific plugin version in multidex test (flutter#93148)

Co-authored-by: Hans Muller <[email protected]>
Co-authored-by: Emmanuel Garcia <[email protected]>
…lutter#93841)

* [flutter_tools] Catch lack of flutter tools source missing (flutter#93168)

* 'Update Engine revision to bcc2b7f for beta release 2.8.0-3.2.pre'

* Replace text directionality control characters with escape sequences in the semantics_tester (flutter#93034)

Co-authored-by: Christopher Fujino <[email protected]>
Co-authored-by: Jason Simmons <[email protected]>
…lutter#94490)

* make CIPD url customizable using FLUTTER_STORAGE_BASE_URL (flutter#94137)

* 'Update Engine revision to 06a7363 for beta release 2.8.0-3.3.pre'

* Update plugin lint test for federated url_launcher plugin (flutter#94374)

Co-authored-by: Yegor <[email protected]>
Co-authored-by: Jenn Magder <[email protected]>
@zlshames
Copy link
Member

Hey, I assume this is whatever was in the upstream's master? Just want to make sure. Also, can you resolve conflicts? Thanks!

@ryanawhelan
Copy link
Author

correct. After my initial commit/PR to add to the test files, I just did a "git pull upstream", "git push origin", and then resolved all the conflicts. It just seemed to all all of that to the existing PR rather than create a 2nd one. Its weird, I am not seeing any conflicts in my repo. Can you tell me what file you are seeing conflicts in?

@ryanawhelan
Copy link
Author

@zlshames sorry, conflicts should be fixed now.

@ryanawhelan ryanawhelan reopened this Jan 15, 2022
@ryanawhelan
Copy link
Author

@googlebot I consent

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants