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

Update lints, require Dart ^3.1 #52

Merged
merged 2 commits into from
Nov 29, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/test-package.yml
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ jobs:
matrix:
# Add macos-latest and/or windows-latest if relevant for this package.
os: [ubuntu-latest]
sdk: [2.17.0, dev]
sdk: [3.1, dev]
steps:
- uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11
- uses: dart-lang/setup-dart@b64355ae6ca0b5d484f0106a033dd1388965d06d
Expand Down
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,7 @@
## 2.1.1-wip

* Require Dart 3.1.

## 2.1.0

- Remove the expectation that the process exits during the normal test body.
Expand Down
15 changes: 2 additions & 13 deletions analysis_options.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
include: package:lints/recommended.yaml
# https://dart.dev/tools/analysis#the-analysis-options-file
include: package:dart_flutter_team_lints/analysis_options.yaml

analyzer:
language:
Expand All @@ -8,23 +9,11 @@ analyzer:

linter:
rules:
- always_declare_return_types
- avoid_unused_constructor_parameters
- cancel_subscriptions
- directives_ordering
- lines_longer_than_80_chars
- literal_only_boolean_expressions
- missing_whitespace_between_adjacent_strings
- no_adjacent_strings_in_list
- no_runtimeType_toString
- omit_local_variable_types
- package_api_docs
- prefer_relative_imports
- prefer_single_quotes
- test_types_in_equals
- throw_in_finally
- type_annotate_public_apis
- unawaited_futures
- unnecessary_await_in_return
- unnecessary_lambdas
- use_super_parameters
6 changes: 3 additions & 3 deletions pubspec.yaml
Original file line number Diff line number Diff line change
@@ -1,11 +1,11 @@
name: test_process
version: 2.1.0
version: 2.1.1-wip
description:
"Test processes: starting; validating stdout and stderr; checking exit code"
repository: https://github.com/dart-lang/test_process

environment:
sdk: ">=2.17.0 <3.0.0"
sdk: ^3.1.0

dependencies:
async: ^2.5.0
Expand All @@ -14,5 +14,5 @@ dependencies:
test: ^1.16.0

dev_dependencies:
lints: ^2.0.0
dart_flutter_team_lints: ^2.0.0
test_descriptor: ^2.0.0
3 changes: 1 addition & 2 deletions test/test_process_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -6,11 +6,10 @@ import 'dart:io';

import 'package:path/path.dart' as p;
import 'package:test/test.dart';

import 'package:test_descriptor/test_descriptor.dart' as d;
import 'package:test_process/test_process.dart';

final throwsTestFailure = throwsA(TypeMatcher<TestFailure>());
final throwsTestFailure = throwsA(isA<TestFailure>());

void main() {
group('shouldExit()', () {
Expand Down