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

[battery_platform_interface] Introduce package. #2975

Merged
merged 2 commits into from
Aug 28, 2020
Merged

[battery_platform_interface] Introduce package. #2975

merged 2 commits into from
Aug 28, 2020

Conversation

yash1200
Copy link
Contributor

Description

In this PR, I have added platform interface package to the battery plugin.

Checklist

Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
  • All existing and new tests are passing.
  • I updated/added relevant documentation (doc comments with ///).
  • The analyzer (flutter analyze) does not report any problems on my PR.
  • I read and followed the Flutter Style Guide.
  • The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
  • I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
  • I updated CHANGELOG.md to add a description of the change.
  • I signed the CLA.
  • I am willing to follow-up on review comments in a timely manner.

Breaking Change

Does your PR require plugin users to manually update their apps to accommodate your change?

  • Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
  • No, this is not a breaking change.

@yash1200
Copy link
Contributor Author

Hey @ditman kindly have a look at this one.

@@ -0,0 +1,11 @@
/// Indicates the current battery state.
Copy link
Member

Choose a reason for hiding this comment

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

Please, don't use camelCase in the name of this file, this file should be: battery_state.dart

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I renamed it now.

@ditman ditman changed the title [battery] Platform interface added. [battery_platform_interface] Introduce package. Aug 28, 2020
@ditman ditman merged commit 161aed2 into flutter:master Aug 28, 2020
@ditman
Copy link
Member

ditman commented Aug 28, 2020

Tagged and published battery_platform_interface: ^1.0.0

@yash1200
Copy link
Contributor Author

Thank You :)

@yash1200 yash1200 deleted the battery branch August 28, 2020 20:58
@yash1200
Copy link
Contributor Author

In porting battery to use platform interface I have to delete the battery/test now. Can I do that ? Because the tests are no in battery_platform_interface.

@ditman
Copy link
Member

ditman commented Aug 28, 2020

@yash1200 you can leave the tests in battery/test. Why do you have to delete them? Because the method channel is not publicly visible anymore?

(Tests in battery/test should call methods in the plugin class, and in the platform_interface they should call directly methods on the method_channel class)

KevinTheGray added a commit to KevinTheGray/plugins that referenced this pull request Aug 31, 2020
* master:
  [url_launcher, url_launcher_web]Added webOnlyWindowName parameter to launch()  (flutter#2979)
  [battery_platform_interface] Introduce package. (flutter#2975)
  [url_launcher_platform_interface] Add webOnlyWindowName parameter to launch() (flutter#2974)
  [share] Add sharing file support (android & ios) (flutter#970)
  [integration_test] add support to get timeline (flutter#2947)
  [battery] Moved battery to its own subdir (flutter#2969)
  [integration_test] Fix channel name for iOS, docs (flutter#2972)
  [integration_test] Show stack trace of widget test errors on the platform side (flutter#2967)
  [video_player] Fix `setWithOthers` test (flutter#2970)
  [video_player_platform_interface] fix `setMixIWithOthers` test channel (flutter#2968)
  Add README instructions for enablePendingPurchases (flutter#2921)
  Rebase previous commits (flutter#2966)
  Avoid walleye 26
  Updating documentation to use isEmpty check. (flutter#2955)
  [integration_test] Minor cleanup (flutter#2956)
  [path_provider, video_player] Tweak pubspecs to pass bulk analysis. (flutter#2959)
  [android_intent] fix platform package dep (flutter#2948)
  [shared_preferences_linux] update `file` package dep version to be compatible with framework (flutter#2950)
jorgefspereira pushed a commit to jorgefspereira/plugins_flutter that referenced this pull request Oct 10, 2020
FlutterSu pushed a commit to FlutterSu/flutter-plugins that referenced this pull request Nov 20, 2020
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