Skip to content

Commit 8e49fbc

Browse files
committed
chore: update linting rules and apply them
1 parent 159ad69 commit 8e49fbc

File tree

5 files changed

+37
-21
lines changed

5 files changed

+37
-21
lines changed
Lines changed: 25 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,29 @@
11
include: package:flutter_lints/flutter.yaml
22

3-
analyzer:
4-
53
linter:
64
rules:
5+
prefer_single_quotes: true
6+
require_trailing_commas: true
7+
always_declare_return_types: true
8+
avoid_catches_without_on_clauses: true
9+
avoid_equals_and_hash_code_on_mutable_classes: true
10+
avoid_types_on_closure_parameters: true
11+
cancel_subscriptions: true
12+
directives_ordering: true
13+
eol_at_end_of_file: true
14+
omit_local_variable_types: true
15+
prefer_asserts_in_initializer_lists: true
16+
prefer_const_constructors: true
17+
prefer_final_in_for_each: true
18+
prefer_final_locals: true
19+
prefer_null_aware_method_calls: true
20+
prefer_null_aware_operators: true
21+
sort_constructors_first: true
22+
sort_unnamed_constructors_first: true
23+
sort_pub_dependencies: true
24+
type_annotate_public_apis: true
25+
unawaited_futures: true
26+
unnecessary_lambdas: true
27+
unnecessary_parenthesis: true
28+
use_named_constants: true
29+
use_super_parameters: true

clipboard_viewer/lib/main.dart

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
import 'package:flutter/material.dart';
22
import 'package:pasteboard/pasteboard.dart';
3-
import 'package:super_clipboard/super_clipboard.dart';
43
// ignore: implementation_imports
54
import 'package:super_clipboard/src/format_conversions.dart';
5+
import 'package:super_clipboard/super_clipboard.dart';
66

77
void main() {
88
runApp(const MyApp());
@@ -54,13 +54,14 @@ class _MyHomePageState extends State<MyHomePage> {
5454
return ListTile(
5555
title: SelectableText(item.$2.toString()),
5656
subtitle: SelectableText(
57-
item.$1.platformFormats.map((e) => "'$e'").join(", ")),
57+
item.$1.platformFormats.map((e) => "'$e'").join(', '),
58+
),
5859
);
5960
},
6061
),
6162
floatingActionButton: FloatingActionButton(
6263
onPressed: () async {
63-
await Pasteboard.writeFiles(["/home/davide"]);
64+
await Pasteboard.writeFiles(['/home/davide']);
6465
return;
6566
// TODO(@HrX03) is this needed?
6667
// ignore: unused_local_variable, dead_code

clipboard_viewer/pubspec.yaml

Lines changed: 5 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,31 +1,23 @@
11
name: clipboard_viewer
22
description: A new Flutter project.
3-
publish_to: 'none'
3+
publish_to: "none"
44
version: 1.0.0+1
55

66
environment:
7-
sdk: '>=3.0.0-313.0.dev <4.0.0'
7+
sdk: ">=3.0.0-313.0.dev <4.0.0"
88

99
dependencies:
10+
cupertino_icons: ^1.0.2
1011
flutter:
1112
sdk: flutter
12-
13-
cupertino_icons: ^1.0.2
13+
pasteboard: ^0.2.0
1414
super_clipboard: ^0.2.3
1515
super_native_extensions: ^0.2.3
16-
pasteboard: ^0.2.0
1716

1817
dev_dependencies:
18+
flutter_lints: ^2.0.0
1919
flutter_test:
2020
sdk: flutter
2121

22-
flutter_lints: ^2.0.0
23-
24-
2522
flutter:
2623
uses-material-design: true
27-
28-
29-
30-
31-

clipboard_viewer/test/widget_test.dart

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,9 @@
1+
import 'package:clipboard_viewer/main.dart';
12
import 'package:flutter/material.dart';
23
import 'package:flutter_test/flutter_test.dart';
34

4-
import 'package:clipboard_viewer/main.dart';
5-
65
void main() {
7-
testWidgets('Counter increments smoke test', (WidgetTester tester) async {
6+
testWidgets('Counter increments smoke test', (tester) async {
87
await tester.pumpWidget(const MyApp());
98

109
expect(find.text('0'), findsOneWidget);

pubspec.yaml

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

77
environment:
88
sdk: ">=3.0.0 <4.0.0"
9+
flutter: ">=3.24.3"
910

1011
dependencies:
1112
collection: ^1.16.0

0 commit comments

Comments
 (0)