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

[package_info]Update README.md to reflect iOS issue. #1703

Merged
merged 8 commits into from
Jul 16, 2019

Conversation

pambrose
Copy link
Contributor

@pambrose pambrose commented Jun 4, 2019

Description

Changes to the version string in 'pubspec.yaml' are not reflected in the package_info value on iOS.

As noted here, package_info on iOS requires the Xcode build folder to be rebuilt after changes to the version string in pubspec.yaml.

This PR does not fix the problem, but warns users about the need to rebuild the Xcode build folder.

Related Issues

No related issues.

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.

@pambrose pambrose requested a review from cyanglaz as a code owner June 4, 2019 15:27
@cyanglaz cyanglaz changed the title Update README.md to reflect iOS issue. [package_info]Update README.md to reflect iOS issue. Jun 6, 2019
@cyanglaz
Copy link
Contributor

cyanglaz commented Jun 6, 2019

Thank you for creating this PR! Please update the description and related issue in this PR.

@pambrose
Copy link
Contributor Author

pambrose commented Jun 7, 2019

I am sorry, but I am not sure what you are asking me to do. I believe you already prefixed the description with "[package_info]" What more needs to be done? Thanks.

@cyanglaz
Copy link
Contributor

cyanglaz commented Jun 7, 2019

@pambrose No worries. So I only prefixed the title of the PR. There is a description and related issue in the PR body that you might want to fill out, so we know what is this PR about.

@pambrose
Copy link
Contributor Author

pambrose commented Jun 7, 2019

Sorry, now I understand what you wanted.

Copy link
Contributor

@cyanglaz cyanglaz left a comment

Choose a reason for hiding this comment

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

LGTM with some nits:
Please also update pubspec and CHANGELOG.md with a new version.

@collinjackson collinjackson self-requested a review July 15, 2019 22:43
@collinjackson collinjackson merged commit f33f695 into flutter:master Jul 16, 2019
@pambrose pambrose deleted the patch-1 branch July 17, 2019 00:02
mithun-mondal pushed a commit to bKash-developer/archived_plugins that referenced this pull request Aug 6, 2019
* Update README.md to reflect iOS issue.

* Add more details to packages/package_info/README.md

Co-Authored-By: Chris Yang <ychris@google.com>
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.

4 participants