[ci] Enable min SDK version checks (#3095)

* Roll tool version

* Enable new check in CI

* Fix violations

* CHANGELOG updates

* Fix changelog

* Address analyzer issues

* Revert flutter_lints changelog
This commit is contained in:
stuartmorgan
2023-01-26 20:43:00 -08:00
committed by GitHub
parent f324abb762
commit b1a7162985
31 changed files with 53 additions and 15 deletions

View File

@ -8,4 +8,4 @@ git fetch origin main
# Pinned version of the plugin tools, to avoid breakage in this repository # Pinned version of the plugin tools, to avoid breakage in this repository
# when pushing updates from flutter/plugins. # when pushing updates from flutter/plugins.
dart pub global activate flutter_plugin_tools 0.13.2 dart pub global activate flutter_plugin_tools 0.13.4

View File

@ -99,7 +99,9 @@ task:
always: always:
format_script: ./script/tool_runner.sh format --fail-on-change format_script: ./script/tool_runner.sh format --fail-on-change
license_script: $PLUGIN_TOOL_COMMAND license-check license_script: $PLUGIN_TOOL_COMMAND license-check
pubspec_script: ./script/tool_runner.sh pubspec-check # The major and minor versions here should match the lowest version
# analyzed in legacy_version_analyze.
pubspec_script: ./script/tool_runner.sh pubspec-check --min-min-flutter-version=3.0.0 --min-min-dart-version=2.17.0
readme_script: readme_script:
- ./script/tool_runner.sh readme-check - ./script/tool_runner.sh readme-check
# Re-run with --require-excerpts, skipping packages that still need # Re-run with --require-excerpts, skipping packages that still need
@ -157,6 +159,7 @@ task:
- name: legacy_version_analyze - name: legacy_version_analyze
depends_on: analyze depends_on: analyze
matrix: matrix:
# Change the arguments to pubspec-check when changing these values.
env: env:
CHANNEL: "3.0.5" CHANNEL: "3.0.5"
DART_VERSION: "2.17.6" DART_VERSION: "2.17.6"

View File

@ -31,7 +31,7 @@ jobs:
with: with:
fetch-depth: 0 # Fetch all history so the tool can get all the tags to determine version. fetch-depth: 0 # Fetch all history so the tool can get all the tags to determine version.
- name: Set up tools - name: Set up tools
run: dart pub global activate flutter_plugin_tools 0.13.2 run: dart pub global activate flutter_plugin_tools 0.13.4
# # This workflow should be the last to run. So wait for all the other tests to succeed. # # This workflow should be the last to run. So wait for all the other tests to succeed.
- name: Wait on all tests - name: Wait on all tests

View File

@ -7,6 +7,7 @@ version: 0.0.1
environment: environment:
sdk: ">=2.12.0 <3.0.0" sdk: ">=2.12.0 <3.0.0"
flutter: ">=3.0.0"
dependencies: dependencies:
animations: animations:

View File

@ -1,3 +1,7 @@
## NEXT
* Updates minimum SDK version to Flutter 3.0.
## 0.3.3+2 ## 0.3.3+2
* Fixes lint warnings in tests. * Fixes lint warnings in tests.

View File

@ -26,6 +26,7 @@ class XFile extends XFileBase {
/// ///
/// `name` needs to be passed from the outside, since it's only available /// `name` needs to be passed from the outside, since it's only available
/// while handling [html.File]s (when the ObjectUrl is created). /// while handling [html.File]s (when the ObjectUrl is created).
// ignore: use_super_parameters
XFile( XFile(
String path, { String path, {
String? mimeType, String? mimeType,

View File

@ -22,14 +22,14 @@ class XFile extends XFileBase {
/// `path` of the file doesn't match what the user sees when selecting it /// `path` of the file doesn't match what the user sees when selecting it
/// (like in web) /// (like in web)
XFile( XFile(
String path, { super.path, {
String? mimeType, String? mimeType,
String? name, String? name,
int? length, int? length,
Uint8List? bytes, Uint8List? bytes,
DateTime? lastModified, DateTime? lastModified,
@visibleForTesting CrossFileTestOverrides? overrides, @visibleForTesting CrossFileTestOverrides? overrides,
}) : super(path) { }) {
throw UnimplementedError( throw UnimplementedError(
'CrossFile is not available in your current platform.'); 'CrossFile is not available in your current platform.');
} }

View File

@ -17,6 +17,7 @@ class XFile extends XFileBase {
/// [bytes] is ignored; the parameter exists only to match the web version of /// [bytes] is ignored; the parameter exists only to match the web version of
/// the constructor. To construct a dart:io XFile from bytes, use /// the constructor. To construct a dart:io XFile from bytes, use
/// [XFile.fromData]. /// [XFile.fromData].
// ignore: use_super_parameters
XFile( XFile(
String path, { String path, {
String? mimeType, String? mimeType,

View File

@ -5,7 +5,7 @@ issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+
version: 0.3.3+2 version: 0.3.3+2
environment: environment:
sdk: ">=2.12.0 <3.0.0" sdk: ">=2.17.0 <3.0.0"
dependencies: dependencies:
js: ^0.6.3 js: ^0.6.3

View File

@ -109,7 +109,7 @@ void main() {
/// An XFile subclass that tracks reads, for testing purposes. /// An XFile subclass that tracks reads, for testing purposes.
class TestXFile extends XFile { class TestXFile extends XFile {
TestXFile(String path) : super(path); TestXFile(super.path);
bool hasBeenRead = false; bool hasBeenRead = false;

View File

@ -4,7 +4,7 @@ description: A project that showcases how to enable the recommended lints for Fl
publish_to: none publish_to: none
environment: environment:
sdk: '>=2.12.0 <3.0.0' sdk: '>=2.17.0 <3.0.0'
# Add the latest version of `package:flutter_lints` as a dev_dependency. The # Add the latest version of `package:flutter_lints` as a dev_dependency. The
# lint set provided by this package is activated in the `analysis_options.yaml` # lint set provided by this package is activated in the `analysis_options.yaml`

View File

@ -5,7 +5,7 @@ issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+
version: 2.0.1 version: 2.0.1
environment: environment:
sdk: '>=2.17.0-0 <3.0.0' sdk: '>=2.17.0 <3.0.0'
dependencies: dependencies:
lints: ^2.0.0 lints: ^2.0.0

View File

@ -1,3 +1,7 @@
## NEXT
* Updates minimum SDK version to Flutter 3.0.
## 4.2.0 ## 4.2.0
* Adds iOS template app icons, updated to square icons with no transparency. * Adds iOS template app icons, updated to square icons with no transparency.

View File

@ -5,4 +5,4 @@ issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+
version: 4.2.0 version: 4.2.0
environment: environment:
sdk: ">=2.12.0 <3.0.0" sdk: ">=2.17.0 <3.0.0"

View File

@ -1,3 +1,7 @@
## NEXT
* Updates minimum SDK version to Flutter 3.0.
## 1.0.6 ## 1.0.6
- Fixes lint warnings. - Fixes lint warnings.

View File

@ -6,7 +6,7 @@ repository: https://github.com/flutter/packages/tree/main/packages/metrics_cente
issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+metrics_center%22 issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+metrics_center%22
environment: environment:
sdk: '>=2.12.0 <3.0.0' sdk: '>=2.17.0 <3.0.0'
dependencies: dependencies:
crypto: ^3.0.1 crypto: ^3.0.1

View File

@ -1,3 +1,7 @@
## NEXT
* Updates minimum SDK version to Flutter 3.0.
## 0.3.2+2 ## 0.3.2+2
* Fixes lints warnings. * Fixes lints warnings.

View File

@ -5,7 +5,7 @@ issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+
version: 0.3.2+2 version: 0.3.2+2
environment: environment:
sdk: ">=2.14.0 <3.0.0" sdk: ">=2.17.0 <3.0.0"
dependencies: dependencies:
meta: ^1.3.0 meta: ^1.3.0

View File

@ -5,6 +5,7 @@ version: 0.1.0
environment: environment:
sdk: ">=2.12.0 <3.0.0" sdk: ">=2.12.0 <3.0.0"
flutter: ">=3.0.0"
dependencies: dependencies:
flutter: flutter:

View File

@ -5,6 +5,7 @@ version: 1.0.0+1
environment: environment:
sdk: ">=2.12.0 <3.0.0" sdk: ">=2.12.0 <3.0.0"
flutter: ">=3.0.0"
dependencies: dependencies:
cupertino_icons: ^1.0.2 cupertino_icons: ^1.0.2

View File

@ -4,6 +4,7 @@ publish_to: none # Remove this line if you wish to publish to pub.dev
version: 1.0.0+1 version: 1.0.0+1
environment: environment:
sdk: ">=2.12.0 <3.0.0" sdk: ">=2.12.0 <3.0.0"
flutter: ">=3.0.0"
dependencies: dependencies:
flutter: flutter:

View File

@ -16,6 +16,7 @@ version: 1.0.0+1
environment: environment:
sdk: ">=2.12.0 <3.0.0" sdk: ">=2.12.0 <3.0.0"
flutter: ">=3.0.0"
dependencies: dependencies:
cupertino_icons: ^1.0.5 cupertino_icons: ^1.0.5

View File

@ -5,6 +5,7 @@ version: 1.0.0+1
environment: environment:
sdk: ">=2.13.0 <3.0.0" sdk: ">=2.13.0 <3.0.0"
flutter: ">=3.0.0"
dependencies: dependencies:
flutter: flutter:

View File

@ -5,6 +5,7 @@ version: 1.0.0+1
environment: environment:
sdk: ">=2.13.0 <3.0.0" sdk: ">=2.13.0 <3.0.0"
flutter: ">=3.0.0"
dependencies: dependencies:
flutter: flutter:

View File

@ -5,6 +5,7 @@ version: 1.0.0+1
environment: environment:
sdk: ">=2.13.0 <3.0.0" sdk: ">=2.13.0 <3.0.0"
flutter: ">=3.0.0"
dependencies: dependencies:
flutter: flutter:

View File

@ -5,6 +5,7 @@ version: 1.0.0+1
environment: environment:
sdk: ">=2.13.0 <3.0.0" sdk: ">=2.13.0 <3.0.0"
flutter: ">=3.0.0"
dependencies: dependencies:
flutter: flutter:

View File

@ -4,7 +4,7 @@ version: 1.0.0
publish_to: none publish_to: none
environment: environment:
sdk: '>=2.14.0 <3.0.0' sdk: '>=2.17.0 <3.0.0'
dependencies: dependencies:
lcov_parser: 0.1.1 lcov_parser: 0.1.1

View File

@ -1,3 +1,7 @@
## NEXT
* Updates minimum SDK version to Flutter 3.0.
## 0.2.0+3 ## 0.2.0+3
* Returns null instead of throwing exception from getUserDirectory when xdg-user-dir executable is missing. * Returns null instead of throwing exception from getUserDirectory when xdg-user-dir executable is missing.

View File

@ -5,7 +5,7 @@ issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+
version: 0.2.0+3 version: 0.2.0+3
environment: environment:
sdk: ">=2.12.0 <3.0.0" sdk: ">=2.17.0 <3.0.0"
platforms: platforms:
linux: linux:

View File

@ -1,3 +1,7 @@
## NEXT
* Updates minimum SDK version to Flutter 3.0.
## 1.0.5 ## 1.0.5
* Updates README to reference correct URL. * Updates README to reference correct URL.

View File

@ -6,7 +6,7 @@ issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+
version: 1.0.5 version: 1.0.5
environment: environment:
sdk: ">=2.12.0 <3.0.0" sdk: ">=2.17.0 <3.0.0"
flutter: flutter:
fonts: fonts: