Skip to content

Commit e29b045

Browse files
committed
Merge branch 'master' into add_zoom_to_android_webview
* master: (76 commits) Add plugin issue query to README (flutter#3493) Automatically add platform labels (flutter#3487) Migrate shared_preferences_platform_interfaces to null safety (flutter#3466) Remove Dart stubs from macOS plugins (flutter#3491) [path_provider_windows] Resolve FFI stabilization changes (flutter#3485) [local_auth] Fix incorrect switch fallthrough (flutter#3473) Remove amirh from CODEOWNERS (flutter#3484) [url_launcher_web] Fix Link misalignment issue (flutter#3476) [ci][image_picker][webviews_flutter] enable Xcode 12 (flutter#3461) [url_launcher] Update description in pubspec.yaml (flutter#2858) [integration_test] Fix tests from changes to `flutter test` machine output (flutter#3480) [package_info] Register IntegrationTestPlugin in the example app. (flutter#3478) [ci] fix ci failure on ios builds (flutter#3470) Reland "[cross_file] Migrate to null-safety. (flutter#3452)" (flutter#3469) Revert "[cross_file] Migrate to null-safety. (flutter#3452)" (flutter#3468) [image_picker_platform_interface] fix test asset file location (flutter#3467) [cross_file] Migrate to null-safety. (flutter#3452) [path_provider] drop uuid (flutter#3465) [cross_file] Use Uri when calling package:http methods (flutter#3462) bump vmservice (flutter#3463) ... # Conflicts: # packages/webview_flutter/CHANGELOG.md
2 parents df9a0a9 + a87497f commit e29b045

File tree

281 files changed

+8787
-1644
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

281 files changed

+8787
-1644
lines changed

.cirrus.yml

Lines changed: 38 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -134,11 +134,12 @@ task:
134134
- xvfb-run ./script/incremental_build.sh drive-examples --linux
135135

136136
task:
137+
# Xcode 12 task
137138
# don't run on release tags since it creates O(n^2) tasks where n is the number of plugins
138139
only_if: $CIRRUS_TAG == ''
139140
use_compute_credits: $CIRRUS_USER_COLLABORATOR == 'true'
140141
osx_instance:
141-
image: catalina-xcode-11.3.1-flutter
142+
image: big-sur-xcode-12.3
142143
upgrade_script:
143144
- sudo gem install cocoapods
144145
- flutter channel stable
@@ -149,7 +150,7 @@ task:
149150
activate_script: pub global activate flutter_plugin_tools
150151
create_simulator_script:
151152
- xcrun simctl list
152-
- xcrun simctl create Flutter-iPhone com.apple.CoreSimulator.SimDeviceType.iPhone-X com.apple.CoreSimulator.SimRuntime.iOS-13-3 | xargs xcrun simctl boot
153+
- xcrun simctl create Flutter-iPhone com.apple.CoreSimulator.SimDeviceType.iPhone-11 com.apple.CoreSimulator.SimRuntime.iOS-14-3 | xargs xcrun simctl boot
153154
matrix:
154155
- name: build_all_plugins_ipa
155156
env:
@@ -162,17 +163,6 @@ task:
162163
- if [[ "$CHANNEL" -eq "stable" ]]; then find . | grep _web$ | xargs rm -rf; fi
163164
- flutter channel $CHANNEL
164165
- ./script/build_all_plugins_app.sh ios --no-codesign
165-
- name: lint_darwin_plugins
166-
env:
167-
matrix:
168-
PLUGIN_SHARDING: "--shardIndex 0 --shardCount 2"
169-
PLUGIN_SHARDING: "--shardIndex 1 --shardCount 2"
170-
script:
171-
# TODO(jmagman): Lint macOS podspecs but skip any that fail library validation.
172-
- find . -name "*.podspec" | xargs grep -l "osx" | xargs rm
173-
# Skip the dummy podspecs used to placate the tool.
174-
- find . -name "*_web*.podspec" -o -name "*_mac*.podspec" | xargs rm
175-
- ./script/incremental_build.sh podspecs
176166
- name: build-ipas+drive-examples
177167
env:
178168
PATH: $PATH:/usr/local/bin
@@ -191,15 +181,49 @@ task:
191181
# https://github.com/flutter/flutter/issues/42864
192182
- if [[ "$CHANNEL" -eq "stable" ]]; then find . | grep _web$ | xargs rm -rf; fi
193183
- flutter channel $CHANNEL
184+
- flutter upgrade
194185
- ./script/incremental_build.sh build-examples --ipa
195186
- ./script/incremental_build.sh drive-examples
196-
- ./script/incremental_build.sh xctest --target RunnerUITests --skip $PLUGINS_TO_SKIP_XCTESTS
187+
- ./script/incremental_build.sh xctest --target RunnerUITests --skip $PLUGINS_TO_SKIP_XCTESTS --ios-destination "platform=iOS Simulator,name=iPhone 11,OS=14.3"
188+
197189
task:
190+
# Xcode 11 task
191+
# TODO(cyanglaz): merge Xcode 11 task to Xcode 12 task when all the matrix can be run in Xcode 12.
198192
# don't run on release tags since it creates O(n^2) tasks where n is the number of plugins
199193
only_if: $CIRRUS_TAG == ''
200194
use_compute_credits: $CIRRUS_USER_COLLABORATOR == 'true'
201195
osx_instance:
202196
image: catalina-xcode-11.3.1-flutter
197+
upgrade_script:
198+
- sudo gem install cocoapods
199+
- flutter channel stable
200+
- flutter upgrade
201+
- flutter channel master
202+
- flutter upgrade
203+
- git fetch origin master
204+
activate_script: pub global activate flutter_plugin_tools
205+
create_simulator_script:
206+
- xcrun simctl list
207+
- xcrun simctl create Flutter-iPhone com.apple.CoreSimulator.SimDeviceType.iPhone-X com.apple.CoreSimulator.SimRuntime.iOS-13-3 | xargs xcrun simctl boot
208+
matrix:
209+
- name: lint_darwin_plugins
210+
env:
211+
matrix:
212+
PLUGIN_SHARDING: "--shardIndex 0 --shardCount 2"
213+
PLUGIN_SHARDING: "--shardIndex 1 --shardCount 2"
214+
script:
215+
# TODO(jmagman): Lint macOS podspecs but skip any that fail library validation.
216+
- find . -name "*.podspec" | xargs grep -l "osx" | xargs rm
217+
# Skip the dummy podspecs used to placate the tool.
218+
- find . -name "*_web*.podspec" -o -name "*_mac*.podspec" | xargs rm
219+
- ./script/incremental_build.sh podspecs
220+
221+
task:
222+
# don't run on release tags since it creates O(n^2) tasks where n is the number of plugins
223+
only_if: $CIRRUS_TAG == ''
224+
use_compute_credits: $CIRRUS_USER_COLLABORATOR == 'true'
225+
osx_instance:
226+
image: big-sur-xcode-12.3
203227
setup_script:
204228
- flutter config --enable-macos-desktop
205229
upgrade_script:

.github/PULL_REQUEST_TEMPLATE.md

Lines changed: 19 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -1,37 +1,32 @@
1-
## Description
1+
*Replace this paragraph with a description of what this PR is changing or adding, and why. Consider including before/after screenshots.*
22

3-
*Replace this paragraph with a description of what this PR is doing. If you're modifying existing behavior, describe the existing behavior, how this PR is changing it, and what motivated the change.*
3+
*List which issues are fixed by this PR. You must list at least one issue.*
44

5-
## Related Issues
5+
*If you had to change anything in the [flutter/tests] repo, include a link to the migration guide as per the [breaking change policy].*
66

7-
*Replace this paragraph with a list of issues related to this PR from the [issue database](https://github.com/flutter/flutter/issues). Indicate, which of these issues are resolved or fixed by this PR. Note that you'll have to prefix the issue numbers with flutter/flutter#.*
8-
9-
## Checklist
10-
11-
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes (`[x]`). This will ensure a smooth and quick review process.
7+
## Pre-launch Checklist
128

9+
- [ ] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. `[shared_preferences]`
1310
- [ ] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
14-
- [ ] My PR includes unit or integration tests for *all* changed/updated/fixed behaviors (See [Contributor Guide]).
15-
- [ ] All existing and new tests are passing.
16-
- [ ] I updated/added relevant documentation (doc comments with `///`).
17-
- [ ] The analyzer (`flutter analyze`) does not report any problems on my PR.
18-
- [ ] I read and followed the [Flutter Style Guide].
19-
- [ ] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
11+
- [ ] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
12+
- [ ] I read and followed the [Flutter Style Guide] and the [C++, Objective-C, Java style guides].
13+
- [ ] I listed at least one issue that this PR fixes in the description above.
14+
- [ ] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test exempt.
2015
- [ ] I updated pubspec.yaml with an appropriate new version according to the [pub versioning philosophy].
2116
- [ ] I updated CHANGELOG.md to add a description of the change.
17+
- [ ] I updated/added relevant documentation (doc comments with `///`).
2218
- [ ] I signed the [CLA].
23-
- [ ] I am willing to follow-up on review comments in a timely manner.
24-
25-
## Breaking Change
26-
27-
Does your PR require plugin users to manually update their apps to accommodate your change?
19+
- [ ] All existing and new tests are passing.
2820

29-
- [ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
30-
- [ ] No, this is *not* a breaking change.
21+
If you need help, consider asking for advice on the #hackers-new channel on [Discord].
3122

3223
<!-- Links -->
33-
[issue database]: https://github.com/flutter/flutter/issues
34-
[Contributor Guide]: https://github.com/flutter/plugins/blob/master/CONTRIBUTING.md
24+
[Contributor Guide]: https://github.com/flutter/flutter/wiki/Tree-hygiene#overview
25+
[Tree Hygiene]: https://github.com/flutter/flutter/wiki/Tree-hygiene
3526
[Flutter Style Guide]: https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo
36-
[pub versioning philosophy]: https://www.dartlang.org/tools/pub/versioning
27+
[C++, Objective-C, Java style guides]: https://github.com/flutter/engine/blob/master/CONTRIBUTING.md#style
3728
[CLA]: https://cla.developers.google.com/
29+
[flutter/tests]: https://github.com/flutter/tests
30+
[breaking change policy]: https://github.com/flutter/flutter/wiki/Tree-hygiene#handling-breaking-changes
31+
[Discord]: https://github.com/flutter/flutter/wiki/Chat
32+
[pub versioning philosophy]: https://dart.dev/tools/pub/versioning

.github/labeler.yml

Lines changed: 110 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,110 @@
1+
'p: android_alarm_manager':
2+
- packages/android_alarm_manager/**/*
3+
4+
'p: android_intent':
5+
- packages/android_intent/**/*
6+
7+
'p: battery':
8+
- packages/battery/**/*
9+
10+
'p: camera':
11+
- packages/camera/**/*
12+
13+
'p: connectivity':
14+
- packages/connectivity/**/*
15+
16+
'p: cross_file':
17+
- packages/cross_file/**/*
18+
19+
'p: device_info':
20+
- packages/device_info/**/*
21+
22+
'p: e2e':
23+
- packages/e2e/**/*
24+
25+
'p: espresso':
26+
- packages/espresso/**/*
27+
28+
'p: file_selector':
29+
- packages/file_selector/**/*
30+
31+
'p: flutter_plugin_android_lifecycle':
32+
- packages/flutter_plugin_android_lifecycle/**/*
33+
34+
'p: google_maps_flutter':
35+
- packages/google_maps_flutter/**/*
36+
37+
'p: google_sign_in':
38+
- packages/google_sign_in/**/*
39+
40+
'p: image_picker':
41+
- packages/image_picker/**/*
42+
43+
'p: in_app_purchase':
44+
- packages/in_app_purchase/**/*
45+
46+
'p: integration_test':
47+
- packages/integration_test/**/*
48+
49+
'p: ios_platform_images':
50+
- packages/ios_platform_images/**/*
51+
52+
'p: local_auth':
53+
- packages/local_auth/**/*
54+
55+
'p: package_info':
56+
- packages/package_info/**/*
57+
58+
'p: path_provider':
59+
- packages/path_provider/**/*
60+
61+
'p: plugin_platform_interface':
62+
- packages/plugin_platform_interface/**/*
63+
64+
'p: quick_actions':
65+
- packages/quick_actions/**/*
66+
67+
'p: sensors':
68+
- packages/sensors/**/*
69+
70+
'p: share':
71+
- packages/share/**/*
72+
73+
'p: shared_preferences':
74+
- packages/shared_preferences/**/*
75+
76+
'p: url_launcher':
77+
- packages/url_launcher/**/*
78+
79+
'p: video_player':
80+
- packages/video_player/**/*
81+
82+
'p: webview_flutter':
83+
- packages/webview_flutter/**/*
84+
85+
'p: wifi_info_flutter':
86+
- packages/wifi_info_flutter/**/*
87+
88+
'platform-android':
89+
- packages/*/*_android/**/*
90+
- packages/**/android/**/*
91+
92+
'platform-ios':
93+
- packages/*/*_ios/**/*
94+
- packages/**/ios/**/*
95+
96+
'platform-linux':
97+
- packages/*/*_linux/**/*
98+
- packages/**/linux/**/*
99+
100+
'platform-macos':
101+
- packages/*/*_macos/**/*
102+
- packages/**/macos/**/*
103+
104+
'platform-web':
105+
- packages/*/*_web/**/*
106+
- packages/**/web/**/*
107+
108+
'platform-windows':
109+
- packages/*/*_windows/**/*
110+
- packages/**/windows/**/*
Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
1+
# This workflow applies labels to pull requests based on the
2+
# paths that are modified in the pull request.
3+
#
4+
# Edit `.github/labeler.yml` to configure labels.
5+
#
6+
# For more information, see: https://github.com/actions/labeler
7+
8+
name: Pull Request Labeler
9+
10+
on:
11+
- pull_request_target
12+
13+
jobs:
14+
label:
15+
runs-on: ubuntu-latest
16+
steps:
17+
- uses: actions/labeler@main
18+
with:
19+
repo-token: "${{ secrets.GITHUB_TOKEN }}"
20+
sync-labels: true

AUTHORS

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,3 +59,5 @@ Kazuki Yamaguchi <[email protected]>
5959
Eitan Schwartz <[email protected]>
6060
Chris Rutkowski <[email protected]>
6161
Juan Alvarez <[email protected]>
62+
Aleksandr Yurkovskiy <[email protected]>
63+
Anton Borries <[email protected]>

CODEOWNERS

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66

77
packages/android_alarm_manager/** @bkonyi
88
packages/android_intent/** @mklim @matthew-carroll
9-
packages/battery/** @amirh @matthew-carroll
9+
packages/battery/** @matthew-carroll
1010
packages/camera/** @bparrishMines
1111
packages/connectivity/** @cyanglaz @matthew-carroll
1212
packages/cross_file/** @ditman @mvanbeusekom
@@ -24,4 +24,3 @@ packages/path_provider/** @matthew-carroll
2424
packages/shared_preferences/** @matthew-carroll
2525
packages/url_launcher/** @mklim
2626
packages/video_player/** @iskakaushik @cyanglaz
27-
packages/webview_flutter/** @amirh

README.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,9 @@ These plugins are also available on
1919
Please file any issues, bugs, or feature requests in the [main flutter
2020
repo](https://github.com/flutter/flutter/issues/new).
2121

22+
Issues pertaining to this repository are [labeled
23+
"plugin"](https://github.com/flutter/flutter/issues?q=is%3Aopen+is%3Aissue+label%3Aplugin).
24+
2225
## Contributing
2326

2427
If you wish to contribute a new plugin to the Flutter ecosystem, please

packages/android_alarm_manager/CHANGELOG.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,7 @@
1+
## 0.4.5+20
2+
3+
* Update the example app: remove the deprecated `RaisedButton` and `FlatButton` widgets.
4+
15
## 0.4.5+19
26

37
* Fix outdated links across a number of markdown files ([#3276](https://github.com/flutter/plugins/pull/3276))

packages/android_alarm_manager/example/lib/main.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,7 @@ class _AlarmHomePageState extends State<_AlarmHomePage> {
131131
),
132132
],
133133
),
134-
RaisedButton(
134+
ElevatedButton(
135135
child: Text(
136136
'Schedule OneShot Alarm',
137137
),

packages/android_alarm_manager/pubspec.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ description: Flutter plugin for accessing the Android AlarmManager service, and
44
# 0.4.y+z is compatible with 1.0.0, if you land a breaking change bump
55
# the version to 2.0.0.
66
# See more details: https://github.com/flutter/flutter/wiki/Package-migration-to-1.0.0
7-
version: 0.4.5+19
7+
version: 0.4.5+20
88
homepage: https://github.com/flutter/plugins/tree/master/packages/android_alarm_manager
99

1010
dependencies:

packages/android_intent/CHANGELOG.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,7 @@
1+
## 2.0.0-nullsafety.2
2+
3+
* Update the example app: remove the deprecated `RaisedButton` and `FlatButton` widgets.
4+
15
## 2.0.0-nullsafety.1
26

37
* Fix outdated links across a number of markdown files ([#3276](https://github.com/flutter/plugins/pull/3276))

0 commit comments

Comments
 (0)