Skip to content

Commit

Permalink
remove version checking (dart-lang/linter#4419)
Browse files Browse the repository at this point in the history
  • Loading branch information
pq authored Jun 1, 2023
1 parent 950e089 commit 58b719a
Show file tree
Hide file tree
Showing 7 changed files with 3 additions and 89 deletions.
5 changes: 2 additions & 3 deletions lib/src/analyzer.dart
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,6 @@ import 'package:analyzer/src/lint/util.dart' as util;
import 'package:analyzer/src/services/lint.dart' as lint_service;
import 'package:pub_semver/pub_semver.dart';

import 'version.dart';

export 'package:analyzer/dart/element/type_system.dart';
export 'package:analyzer/src/dart/ast/token.dart';
export 'package:analyzer/src/dart/element/inheritance_manager3.dart'
Expand Down Expand Up @@ -94,7 +92,8 @@ class Analyzer {

/// Cache linter version; used in summary signatures.
void cacheLinterVersion() {
lint_service.linterVersion = version;
// todo(pq): remove (https://github.com/dart-lang/linter/issues/4418)
lint_service.linterVersion = '1.35.0';
}

/// Create a library name prefix based on [libraryPath], [projectRoot] and
Expand Down
6 changes: 0 additions & 6 deletions lib/src/version.dart

This file was deleted.

2 changes: 0 additions & 2 deletions test/all.dart
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,6 @@ import 'validate_rule_description_format_test.dart'
as validate_rule_description_format;
import 'verify_checks_test.dart' as verify_checks;
import 'verify_reflective_test_suites.dart' as verify_reflective_test_suites;
import 'version_test.dart' as version_test;

void main() {
// Redirect output.
Expand All @@ -45,5 +44,4 @@ void main() {
validate_rule_description_format.main();
verify_checks.main();
verify_reflective_test_suites.main();
version_test.main();
}
10 changes: 0 additions & 10 deletions test/rule_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@ import 'package:analyzer/src/analysis_options/analysis_options_provider.dart';
import 'package:analyzer/src/generated/engine.dart';
import 'package:analyzer/src/lint/io.dart';
import 'package:analyzer/src/lint/registry.dart';
import 'package:analyzer/src/services/lint.dart' as lint_service;
import 'package:analyzer/src/task/options.dart';
import 'package:analyzer/src/utilities/legacy.dart';
import 'package:linter/src/analyzer.dart';
Expand All @@ -24,7 +23,6 @@ import 'package:linter/src/rules/package_prefixed_library_names.dart';
import 'package:linter/src/test_utilities/annotation.dart';
import 'package:linter/src/test_utilities/test_resource_provider.dart';
import 'package:linter/src/utils.dart';
import 'package:linter/src/version.dart';
import 'package:path/path.dart' as p;
import 'package:test/test.dart';

Expand All @@ -36,7 +34,6 @@ import 'util/test_utils.dart';
void main() {
group('rule tests', () {
setUp(setUpSharedTestEnvironment);
defineSanityTests();
defineRuleTests();
experiment_tests.main();
defineRuleUnitTests();
Expand Down Expand Up @@ -182,13 +179,6 @@ void defineRuleUnitTests() {
});
}

/// Test framework sanity.
void defineSanityTests() {
test('linter version caching', () {
expect(lint_service.linterVersion, version);
});
}

/// Handy for debugging.
void defineSoloRuleTest(String ruleToTest) {
for (var entry in Directory(ruleTestDataDir).listSync()) {
Expand Down
28 changes: 0 additions & 28 deletions test/version_test.dart

This file was deleted.

1 change: 1 addition & 0 deletions tool/bot.sh
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@
# Fast fail the script on failures.
set -e


if [ "$LINTER_BOT" = "benchmark" ]; then
echo "Running the linter benchmark..."

Expand Down
40 changes: 0 additions & 40 deletions tool/bot/version_check.dart

This file was deleted.

0 comments on commit 58b719a

Please sign in to comment.