Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

Use package import to import files inside lib/ directory. #2236

Merged
merged 1 commit into from
Oct 25, 2019

Conversation

chingjun
Copy link
Contributor

This is causing issues intenally.

This is also the recommended way in Dart guidelines: https://dart.dev/guides/libraries/create-library-packages#importing-library-files

@chingjun chingjun requested a review from amirh October 25, 2019 22:31
@@ -6,8 +6,8 @@ import 'package:mockito/mockito.dart';
import 'package:flutter/services.dart';
import 'package:flutter_test/flutter_test.dart';

import '../lib/method_channel_url_launcher.dart';
import '../lib/url_launcher_platform_interface.dart';
import 'package:url_launcher_platform_interface/method_channel_url_launcher.dart';
Copy link
Contributor

Choose a reason for hiding this comment

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

interesting, Android Studio kept replacing the package path with '../' when I was editing this file.

Copy link
Contributor

@amirh amirh left a comment

Choose a reason for hiding this comment

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

Can you update the changelog and bump the patch version as well?

@chingjun
Copy link
Contributor Author

Done, PTAL. Thanks.

Copy link
Contributor

@amirh amirh left a comment

Choose a reason for hiding this comment

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

LGTM

@chingjun chingjun merged commit a5b7ba8 into flutter:master Oct 25, 2019
mormih pushed a commit to mormih/plugins that referenced this pull request Nov 17, 2019
sungmin-park pushed a commit to sungmin-park/flutter-plugins that referenced this pull request Dec 17, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants