Skip to content
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

chore(release): Prep for 1.0.0 release #495

Merged
merged 4 commits into from
May 27, 2020
Merged

Conversation

palpatim
Copy link
Member

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@royjit
Copy link
Contributor

royjit commented May 27, 2020

AWSCognitoAuthPlugin Info.plist is missing

Copy link
Contributor

@drochetti drochetti left a comment

Choose a reason for hiding this comment

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

🚀 this is huuuuge!

CHANGELOG.md Outdated

### Bug Fixes

* Tools: Add npx to amplify-app and fix typo - fixes #486 (#490) ([#486](https://github.com/aws-amplify/amplify-ios/issues/486)) ([c7d11a7](https://github.com/aws-amplify/amplify-ios/commit/c7d11a7b1291a2aa588fdcca5bf51e259490d9b5))
Copy link
Contributor

Choose a reason for hiding this comment

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

The numbers are not linked 486, 490

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks, fixed by linking 486

@palpatim
Copy link
Member Author

AWSCognitoAuthPlugin Info.plist is missing

The Info.plist didn't have a correct version before--looks missed it in RC1, too. Let's remember that next time we rename targets. :(

@palpatim palpatim merged commit a8b994b into master May 27, 2020
@palpatim palpatim deleted the palpatim/release-1.0.0 branch May 27, 2020 01:02
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.

4 participants