-
Notifications
You must be signed in to change notification settings - Fork 9.7k
[path_provider] Move package into a path_provider directory #2542
Conversation
|
@franciscojma86 can you edit the PR description to describe what "First step of federation" is, and how many steps there will be in total? |
| name: path_provider | ||
| description: Flutter plugin for getting commonly used locations on the Android & | ||
| iOS file systems, such as the temp and app data directories. | ||
| homepage: https://github.com/flutter/plugins/tree/master/packages/path_provider |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The homepage also changes, to https://github.com/flutter/plugins/tree/master/packages/path_provider/path_provider
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(you can push this minor change when you come back to this file when doing the federation of _macos, just to conserve your CI/CD luck ;) )
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did here :)
99d246b to
4f5bcc1
Compare
…2542) * Move path provider to dir * Fix example * Add dart file * Rename file * Update homepage link
…2542) * Move path provider to dir * Fix example * Add dart file * Rename file * Update homepage link
Description
First step for federation, as outlined in flutter/flutter#46307 (comment)
Related Issues
flutter/flutter#46307
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.///).flutter analyze) does not report any problems on my PR.Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?