Skip to content

bump Dart SDK min version#767

Merged
sensuikan1973 merged 4 commits into
mainfrom
bump_dart_sdk_min_version
May 21, 2025
Merged

bump Dart SDK min version#767
sensuikan1973 merged 4 commits into
mainfrom
bump_dart_sdk_min_version

Conversation

@sensuikan1973
Copy link
Copy Markdown
Owner

No description provided.

@sensuikan1973 sensuikan1973 enabled auto-merge (squash) May 21, 2025 00:11
Comment thread pubspec.yaml
analyzer: ^7.4.5
coverage: ^1.13.1 # See: https://github.com/dart-lang/test/issues/1265
ffigen: ^18.1.0
ffigen: ^19.0.0
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://pub.dev/packages/ffigen/changelog#1900

Breaking change: Major change to the way ObjC methods are code-genned. Methods now use named parameters, making them more readable and closer to how they're written in ObjC. For example, the NSData method dataWithBytes:length: used to be generated as dataWithBytes_length_(Pointer bytes, int length), but is now generated as dataWithBytes(Pointer bytes, {required int length}). Protocol methods are not affected.
Migration tip: A quick way to find affected methods is to search for _(.

@sensuikan1973 sensuikan1973 disabled auto-merge May 21, 2025 00:12
@codecov
Copy link
Copy Markdown

codecov Bot commented May 21, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.41%. Comparing base (e444f7c) to head (620c123).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #767      +/-   ##
==========================================
+ Coverage   96.39%   96.41%   +0.01%     
==========================================
  Files          15       15              
  Lines         666      669       +3     
==========================================
+ Hits          642      645       +3     
  Misses         24       24              
Flag Coverage Δ
macos-latest 96.26% <100.00%> (+0.01%) ⬆️
ubuntu-latest 96.11% <100.00%> (+0.01%) ⬆️
unit_test 96.41% <100.00%> (+0.01%) ⬆️
windows-latest 96.41% <100.00%> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
lib/src/board.dart 100.00% <100.00%> (ø)
lib/src/ffi/dylib_utils.dart 90.00% <100.00%> (+2.50%) ⬆️
lib/src/position.dart 96.66% <100.00%> (+0.11%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sensuikan1973 sensuikan1973 merged commit e89dfa0 into main May 21, 2025
16 checks passed
@sensuikan1973 sensuikan1973 deleted the bump_dart_sdk_min_version branch May 21, 2025 00:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant