Skip to content

Commit

Permalink
Versioning checks update
Browse files Browse the repository at this point in the history
  • Loading branch information
natebiggs committed Jan 24, 2025
1 parent ddfbb37 commit 3a7c791
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 7 deletions.
14 changes: 10 additions & 4 deletions pkgs/dart_services/lib/src/sdk.dart
Original file line number Diff line number Diff line change
Expand Up @@ -158,11 +158,17 @@ final class Sdk {
return true;
}

static final _dartMinorVersionMatch = RegExp(r'([0-9]+.[0-9]+)');
static final _dartVersionMatch = RegExp(r'([0-9]+).([0-9]+)');

double get dartMinorVersion {
int get dartMajorVersion {
final dartVersionString =
_dartMinorVersionMatch.firstMatch(dartVersion)!.group(1)!;
return double.parse(dartVersionString);
_dartVersionMatch.firstMatch(dartVersion)!.group(1)!;
return int.parse(dartVersionString);
}

int get dartMinorVersion {
final dartVersionString =
_dartVersionMatch.firstMatch(dartVersion)!.group(2)!;
return int.parse(dartVersionString);
}
}
2 changes: 1 addition & 1 deletion pkgs/dart_services/test/compiling_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -149,7 +149,7 @@ void defineTests() {
endpoint: (source) => compiler.compileDDC(source),
expectDeltaDill: false,
compiledIndicator: "define('dartpad_main', [");
if (sdk.dartMinorVersion >= 3.8) {
if (sdk.dartMajorVersion >= 3 && sdk.dartMinorVersion >= 8) {
// DDC only supports these at version 3.8 and higher.
testDDCEndpoint('compileNewDDC',
endpoint: (source) => compiler.compileNewDDC(source),
Expand Down
2 changes: 1 addition & 1 deletion pkgs/dart_services/tool/grind.dart
Original file line number Diff line number Diff line change
Expand Up @@ -240,7 +240,7 @@ Future<String> _buildStorageArtifacts(
copy(joinFile(dir, ['flutter_web.dill']), artifactsDir);

// We only expect these hot reload artifacts to work at version 3.8 and later.
if (sdk.dartMinorVersion >= 3.8) {
if (sdk.dartMajorVersion >= 3 && sdk.dartMinorVersion >= 8) {
final argumentsNew = <String>[
path.join(sdk.dartSdkPath, 'bin', 'snapshots', 'dartdevc.dart.snapshot'),
'-s',
Expand Down
1 change: 0 additions & 1 deletion pkgs/dartpad_shared/lib/util.dart

This file was deleted.

0 comments on commit 3a7c791

Please # to comment.