Skip to content

Update testExecutable signature in documentation - #245

Merged
polina-c merged 1 commit into
dart-lang:mainfrom
ValentinVignal:Update-test-executable-signature
Sep 28, 2024
Merged

polina-c merged 1 commit into
dart-lang:mainfrom
ValentinVignal:Update-test-executable-signature

Conversation

@ValentinVignal

Copy link
Copy Markdown
Contributor

In flutter/packages#7546 I discovered that the signature of testExecutable should be

Future<void> testExecutable(FutureOr<void> Function() testMain)

to be able to compile the tests for the chrome platform.

This PR updates the documentation.


  • I’ve reviewed the contributor guide and applied the relevant portions to this PR.
Contribution guidelines:

Note that many Dart repos have a weekly cadence for reviewing PRs - please allow for some latency before initial review feedback.

@ValentinVignal

Copy link
Copy Markdown
Contributor Author

cc @polina-c

@ValentinVignal

Copy link
Copy Markdown
Contributor Author

I believe this doesn't need a CHANGELOG entry/version bump/tests ?

@auto-submit

auto-submit Bot commented Sep 9, 2024

Copy link
Copy Markdown

auto label is removed for dart-lang/leak_tracker/245, due to This PR has not met approval requirements for merging. The PR author is not a member of dart-team and needs 1 more review(s) in order to merge this PR.

  • Merge guidelines: A PR needs at least one approved review if the author is already part of flutter-hackers or two member reviews if the author is not a flutter-hacker before re-applying the autosubmit label. Reviewers: If you left a comment approving, please use the "approve" review action instead.

@polina-c

polina-c commented Sep 9, 2024

Copy link
Copy Markdown
Contributor

Yes, no changelog is needed.
Thank you!

@ValentinVignal

Copy link
Copy Markdown
Contributor Author

Is there something to do for this PR ?

@polina-c
polina-c merged commit e8a9d14 into dart-lang:main Sep 28, 2024
@polina-c

Copy link
Copy Markdown
Contributor

Merged. Thank you and sorry for delays!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants