Skip to content
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

Use dart test on non-Flutter projects #1506

Merged
merged 3 commits into from Mar 25, 2023

Conversation

parlough
Copy link
Member

@parlough parlough commented Mar 24, 2023

A change in test_api caused an incompatibility with the pinned version of flutter_test. I guess projects should explicitly depend on flutter_test or if not a Flutter project, use dart test.

Read dart-lang/test#1977 for more context.

I'm not sure if checking for flutter: is the best way to decide which command to use, so let me know if you think there's a better way or if we should just use a list of projects to use dart commands on :)

@parlough parlough changed the title [Ignore] Use 'dart test' on non-Flutter projects Use dart test on non-Flutter projects Mar 24, 2023
@domesticmouse
Copy link
Contributor

I like this approach as it requires less ongoing maintenance. Assuming we can get it to work 😏

@parlough parlough marked this pull request as ready for review March 25, 2023 00:08
@parlough
Copy link
Member Author

Looks like it's working as expected when reading through the logs. I'll merge this and I can monitor if any issues pop-up overtime, but this should at least put us in a better position for Dart projects!

@parlough parlough merged commit b055b20 into flutter:main Mar 25, 2023
8 checks passed
@parlough parlough deleted the fix/use-dart-test branch March 25, 2023 01:13
domesticmouse added a commit to domesticmouse/samples that referenced this pull request Mar 25, 2023
@domesticmouse domesticmouse mentioned this pull request Mar 25, 2023
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants