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

refactor: Bump min required dart version #187

Merged
merged 2 commits into from
Feb 1, 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/docs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ jobs:
docs:
if: |
github.event_name == 'workflow_dispatch'
|| (github.event.pull_request.merged && contains(github.ref, 'docs/'))
|| (github.event.pull_request.merged && startsWith(github.head_ref, 'docs/'))
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
Expand Down
5 changes: 3 additions & 2 deletions melos.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,8 @@ scripts:
run: melos exec --ignore "*common*" --ignore "*application*" --ignore "*dependency*" -- "dart $MELOS_ROOT_PATH/utils/unused_port.dart | dart pub global run coverage:test_with_coverage --port -"
select-pacakge:
no-private: true
analyze: melos exec -- dart pub global run tuneup check
analyze: melos exec -- dart analyze .
upgrade: melos exec -- dart pub upgrade
activate-all:
run: melos exec -- "dart pub global activate '\$MELOS_PACKAGE_NAME'"
select-pacakge:
Expand All @@ -46,5 +47,5 @@ scripts:
select-pacakge:
no-private: true
environment:
sdk: ">=2.18.6 <3.0.0"
sdk: ">=2.19.0 <3.0.0"

4 changes: 1 addition & 3 deletions packages/cli/analysis_options.yaml
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
include: package:lint/analysis_options.yaml
include: package:lints/recommended.yaml

analyzer:
strong-mode:
implicit-casts: false
exclude:
- tmp/**
- templates/**
1 change: 0 additions & 1 deletion packages/cli/lib/src/cli/scripts/openapi_builder.dart
Original file line number Diff line number Diff line change
Expand Up @@ -170,7 +170,6 @@ Future<Map<String, dynamic>> documentProject(
"licenseName": options.licenseName,
"resolveRelativeUrls": options.resolveRelativeUrls
};
print(project.projectSpecificationFile.readAsStringSync());

final result = await IsolateExecutor.run(
OpenAPIBuilder(variables),
Expand Down
3 changes: 1 addition & 2 deletions packages/cli/pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ version: 4.2.0
homepage: https://github.com/conduit-dart/conduit
description: A modern HTTP server application framework, ORM and OAuth2 provider with OpenAPI 3.0 integration. Foundation for REST, RPC or GraphQL services.
environment:
sdk: ">=2.18.6 <3.0.0"
sdk: ">=2.19.0 <3.0.0"
dependencies:
analyzer: ^5.1.0
args: ^2.1.1
Expand Down Expand Up @@ -32,7 +32,6 @@ dev_dependencies:
fs_test_agent:
path: ../fs_test_agent
http: ^0.13.0
lint: ^2.0.0
lints: ^2.0.1
matcher: ^0.12.12
mockito: ^5.3.2
Expand Down
3 changes: 0 additions & 3 deletions packages/cli/templates/db/analysis_options.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,3 @@
analyzer:
strong-mode:
implicit-casts: false

linter:
rules:
Expand Down
2 changes: 1 addition & 1 deletion packages/cli/templates/db/pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ version: 0.0.1
publish_to: none

environment:
sdk: ">=2.18.6 <3.0.0"
sdk: ">=2.19.0 <3.0.0"

dependencies:
conduit: ^4.1.6
Expand Down
4 changes: 0 additions & 4 deletions packages/cli/templates/db_and_auth/analysis_options.yaml
Original file line number Diff line number Diff line change
@@ -1,7 +1,3 @@
analyzer:
strong-mode:
implicit-casts: false

linter:
rules:
- always_declare_return_types
Expand Down
2 changes: 1 addition & 1 deletion packages/cli/templates/db_and_auth/pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ version: 0.0.1
publish_to: none

environment:
sdk: ">=2.18.6 <3.0.0"
sdk: ">=2.19.0 <3.0.0"

dependencies:
conduit: ^4.1.6
Expand Down
4 changes: 0 additions & 4 deletions packages/cli/templates/default/analysis_options.yaml
Original file line number Diff line number Diff line change
@@ -1,7 +1,3 @@
analyzer:
strong-mode:
implicit-casts: false

linter:
rules:
- always_declare_return_types
Expand Down
2 changes: 1 addition & 1 deletion packages/cli/templates/default/pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ version: 0.0.1
publish_to: none

environment:
sdk: ">=2.18.6 <3.0.0"
sdk: ">=2.19.0 <3.0.0"

dependencies:
conduit: ^4.1.6
Expand Down
2 changes: 1 addition & 1 deletion packages/codable/analysis_options.yaml
Original file line number Diff line number Diff line change
@@ -1 +1 @@
include: package:lint/analysis_options.yaml
include: package:lints/recommended.yaml
4 changes: 2 additions & 2 deletions packages/codable/pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,9 +3,9 @@ version: 4.2.0
homepage: https://github.com/conduit-dart/conduit-codable
description: A serialization library for converting dynamic, structured data (JSON, YAML) into Dart types.
environment:
sdk: '>=2.18.6 <3.0.0'
sdk: '>=2.19.0 <3.0.0'
dependencies:
meta: ^1.3.0
dev_dependencies:
lint: ^2.0.0
lints: ^2.0.1
test: ^1.21.6
2 changes: 1 addition & 1 deletion packages/common/analysis_options.yaml
Original file line number Diff line number Diff line change
@@ -1 +1 @@
include: package:lint/analysis_options.yaml
include: package:lints/recommended.yaml
4 changes: 2 additions & 2 deletions packages/common/pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,8 @@ version: 4.2.0
description: Common classes shared by conduit projects.
repository: https://github.com/bsutton/conduit-orm
environment:
sdk: '>=2.18.6 <3.0.0'
sdk: '>=2.19.0 <3.0.0'
dependencies:
conduit_open_api: ^4.2.0
dev_dependencies:
lint: ^2.0.0
lints: ^2.0.1
99 changes: 1 addition & 98 deletions packages/common_test/analysis_options.yaml
Original file line number Diff line number Diff line change
@@ -1,98 +1 @@
analyzer:
strong-mode:
implicit-casts: false

linter:
rules:
- always_declare_return_types
- always_put_control_body_on_new_line
- always_put_required_named_parameters_first
- always_require_non_null_named_parameters
- annotate_overrides
- avoid_bool_literals_in_conditional_expressions
- avoid_double_and_int_checks
- avoid_empty_else
- avoid_field_initializers_in_const_classes
- avoid_init_to_null
- avoid_null_checks_in_equality_operators
- avoid_positional_boolean_parameters
- avoid_relative_lib_imports
- avoid_renaming_method_parameters
- avoid_return_types_on_setters
- avoid_returning_null
- avoid_single_cascade_in_expression_statements
- avoid_slow_async_io
- avoid_types_as_parameter_names
- avoid_unused_constructor_parameters
- await_only_futures
- camel_case_types
- cancel_subscriptions
- close_sinks
- comment_references
- constant_identifier_names
- control_flow_in_finally
- directives_ordering
- empty_catches
- empty_constructor_bodies
- empty_statements
- file_names
- hash_and_equals
- implementation_imports
- iterable_contains_unrelated_type
- join_return_with_assignment
- library_names
- library_prefixes
- list_remove_unrelated_type
- literal_only_boolean_expressions
- no_duplicate_case_values
- non_constant_identifier_names
- null_closures
- package_api_docs
- package_names
- package_prefixed_library_names
- parameter_assignments
- prefer_adjacent_string_concatenation
- prefer_asserts_in_initializer_lists
- prefer_collection_literals
- prefer_conditional_assignment
- prefer_const_constructors
- prefer_const_constructors_in_immutables
- prefer_const_declarations
- prefer_const_literals_to_create_immutables
- prefer_constructors_over_static_methods
- prefer_contains
- prefer_equal_for_default_values
- prefer_final_fields
- prefer_final_locals
- prefer_foreach
- prefer_generic_function_type_aliases
- prefer_interpolation_to_compose_strings
- prefer_is_empty
- prefer_is_not_empty
- prefer_iterable_whereType
- prefer_typing_uninitialized_variables
- recursive_getters
- slash_for_doc_comments
- sort_constructors_first
- sort_unnamed_constructors_first
- test_types_in_equals
- throw_in_finally
- type_annotate_public_apis
- type_init_formals
- unawaited_futures
- unnecessary_const
- unnecessary_getters_setters
- unnecessary_lambdas
- unnecessary_new
- unnecessary_null_aware_assignments
- unnecessary_null_in_if_null_operators
- unnecessary_overrides
- unnecessary_parenthesis
- unnecessary_statements
- unnecessary_this
- unrelated_type_equality_checks
- use_rethrow_when_possible
- use_string_buffers
- use_to_and_as_if_applicable
- valid_regexps
- void_checks
include: package:lints/recommended.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ class PostgresTestConfig {

PostgresTestConfig._internal();

static late final PostgresTestConfig _self = PostgresTestConfig._internal();
static final PostgresTestConfig _self = PostgresTestConfig._internal();

String get connectionUrl =>
"postgres://$username:$password@$host:$port/$dbName";
Expand Down Expand Up @@ -98,15 +98,15 @@ class PostgresTestConfig {
int get port {
if (_port == null) {
/// Check for an environment variable.
const _key = 'POSTGRES_PORT';
if (Platform.environment.containsKey(_key)) {
final value = Platform.environment[_key];
const key = 'POSTGRES_PORT';
if (Platform.environment.containsKey(key)) {
final value = Platform.environment[key];
if (value != null) {
_port = int.tryParse(value);
}
if (_port == null) {
throw ArgumentError(
"The Environment Variable $_key does not contain a valid integer. Found: $value");
"The Environment Variable $key does not contain a valid integer. Found: $value");
}
}
}
Expand Down
6 changes: 4 additions & 2 deletions packages/common_test/pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,9 @@ homepage: https://github.com/conduit-dart/common-test
description: A collection of classes and function used for testing conduit and by the conduit test harness.
publish_to: none
environment:
sdk: ">=2.18.6 <3.0.0"
sdk: ">=2.19.0 <3.0.0"
dependencies:
conduit_common: ^4.2.0
conduit_core: ^4.2.0
conduit_core: ^4.2.0
dev_dependencies:
lints: ^2.0.1
2 changes: 1 addition & 1 deletion packages/config/analysis_options.yaml
Original file line number Diff line number Diff line change
@@ -1 +1 @@
include: package:lint/analysis_options.yaml
include: package:lints/recommended.yaml
4 changes: 2 additions & 2 deletions packages/config/pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,11 @@ version: 4.2.0
description: A safe and convenient way to read YAML configuration files.
repository: https://github.com/conduit-dart/conduit-config.git
environment:
sdk: '>=2.18.6 <3.0.0'
sdk: '>=2.19.0 <3.0.0'
dependencies:
conduit_runtime: ^4.2.0
meta: ^1.3.0
yaml: ^3.1.0
dev_dependencies:
lint: ^2.0.0
lints: ^2.0.1
test: ^1.21.6
4 changes: 1 addition & 3 deletions packages/core/analysis_options.yaml
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
include: package:lint/analysis_options.yaml
include: package:lints/recommended.yaml

analyzer:
strong-mode:
implicit-casts: false
exclude:
- tmp/**
- templates/**
3 changes: 1 addition & 2 deletions packages/core/pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ version: 4.2.0
homepage: https://github.com/conduit-dart/conduit
description: This is the core of the framework.
environment:
sdk: ">=2.18.6 <3.0.0"
sdk: ">=2.19.0 <3.0.0"
dependencies:
analyzer: ^5.1.0
args: ^2.1.1
Expand Down Expand Up @@ -31,7 +31,6 @@ dev_dependencies:
fs_test_agent:
path: ../fs_test_agent
http: ^0.13.0
lint: ^2.0.0
lints: ^2.0.1
matcher: ^0.12.12
mockito: ^5.3.2
Expand Down
2 changes: 1 addition & 1 deletion packages/fs_test_agent/analysis_options.yaml
Original file line number Diff line number Diff line change
@@ -1 +1 @@
include: package:lint/analysis_options.yaml
include: package:lints/recommended.yaml
15 changes: 8 additions & 7 deletions packages/fs_test_agent/lib/dart_project_agent.dart
Original file line number Diff line number Diff line change
Expand Up @@ -85,10 +85,7 @@ class DartProjectAgent extends WorkingDirectoryAgent {
}

final _analysisOptionsContents = """
analyzer:
strong-mode:
implicit-casts: false
""";
""";

static String _asYaml(Map<String, dynamic> m, {int indent = 0}) {
final buf = StringBuffer();
Expand All @@ -112,15 +109,19 @@ class DartProjectAgent extends WorkingDirectoryAgent {
return buf.toString();
}

String _pubspecContents(String name, Map<String, dynamic> deps,
Map<String, dynamic> devDeps, Map<String, dynamic> dependencyOverrides) {
String _pubspecContents(
String name,
Map<String, dynamic> deps,
Map<String, dynamic> devDeps,
Map<String, dynamic> dependencyOverrides,
) {
return """
name: $name
description: desc
version: 0.0.1

environment:
sdk: ">=2.18.6 <3.0.0"
sdk: ">=2.19.0 <3.0.0"

dependencies:
${_asYaml(deps, indent: 1)}
Expand Down
4 changes: 2 additions & 2 deletions packages/fs_test_agent/pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,10 @@ version: 4.2.0
homepage: https://stablekernel.com
description: Utilities for writing tests to validate file system and Dart project directory operations.
environment:
sdk: '>=2.18.6 <3.0.0'
sdk: '>=2.19.0 <3.0.0'
dependencies:
path: ^1.0.0
pubspec: ^2.0.1
dev_dependencies:
lint: ^2.0.0
lints: ^2.0.1
test: ^1.21.6
6 changes: 1 addition & 5 deletions packages/isolate_exec/analysis_options.yaml
Original file line number Diff line number Diff line change
@@ -1,5 +1 @@
include: package:lint/analysis_options.yaml

analyzer:
strong-mode:
implicit-casts: true
include: package:lints/recommended.yaml
Loading