Skip to content

test(client): remove mirror #314

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

Closed
wants to merge 1 commit into from

Conversation

truongsinh
Copy link
Collaborator

dart test, either in flutter or web browser, does not support mirror,
see dart-lang/sdk#30538 and
flutter/flutter#1150.
Using Directory.current.path might still be problematic for web browser later on,
but now let's focus on flutter only.

dart test, either in flutter or web browser, does not support mirror,
see dart-lang/sdk#30538 and
flutter/flutter#1150.
Using `Directory.current.path` might still be problematic for web browser later on,
but now let's focus on flutter only.
Copy link
Collaborator

@micimize micimize left a comment

Choose a reason for hiding this comment

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

We could use package:file

@micimize
Copy link
Collaborator

@HofmannZ lack of pubspec.lock seems to be breaking CI

@truongsinh
Copy link
Collaborator Author

closing in favor of #323 .

@truongsinh truongsinh closed this Jun 2, 2019
@truongsinh truongsinh deleted the chore/remove-mirror branch June 2, 2019 14:16
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