Skip to content

use dart pub run for the server to speed up tests. #123

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
May 12, 2025
Merged

Conversation

jakemac53
Copy link
Contributor

Also updates the readme with pub global activate instructions as opposed to suggesting you must use the compiled exe. This makes development a lot easier (don't have to recompile manually all the time).

Closes #44

Copy link

PR Health

Changelog Entry ✔️
Package Changed Files

Changes to files need to be accounted for in their respective changelogs.

@auto-submit auto-submit bot removed the autosubmit label May 12, 2025
Copy link

auto-submit bot commented May 12, 2025

autosubmit label was removed for dart-lang/ai/123, because - The status or check suite build (stable) has failed. Please fix the issues identified (or deflake) before re-applying this label.

@auto-submit auto-submit bot removed the autosubmit label May 12, 2025
Copy link

auto-submit bot commented May 12, 2025

autosubmit label was removed for dart-lang/ai/123, because - The status or check suite build (master) has failed. Please fix the issues identified (or deflake) before re-applying this label.

@jakemac53 jakemac53 changed the title use resident compiler and dart run to speed up tests. use dart pub run for the server to speed up tests. May 12, 2025
@jakemac53
Copy link
Contributor Author

The resident compiler seems to fail on GitHub actions... just using dart pub run now which gives a similar speedup just by providing previous compilation results into the next execution.

@jakemac53 jakemac53 merged commit f7ef955 into main May 12, 2025
13 checks passed
@jakemac53 jakemac53 deleted the faster-tests branch May 12, 2025 20:40
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.

[productivity] Update test harness to run faster
2 participants