Skip to content

Update Crashlytics CHANGELOG for beta.3 out of band release - #4788

Merged
samedson merged 5 commits into
masterfrom
samedson-patch-1
Jan 31, 2020
Merged

Update Crashlytics CHANGELOG for beta.3 out of band release#4788
samedson merged 5 commits into
masterfrom
samedson-patch-1

Conversation

@samedson

@samedson samedson commented Jan 31, 2020

Copy link
Copy Markdown
Contributor

Crashlytics is doing an out of band release for beta.3 to fix #4786

Later, I will open a PR for the following changes that will be released with the other Firebase SDKs in a following release:

v4.0.0-beta.4

@samedson samedson changed the title Update Crashlytics CHANGELOG for #4786 Update Crashlytics CHANGELOG for beta.3 out of band release Jan 31, 2020
Comment thread Crashlytics/CHANGELOG.md Outdated
Comment thread Crashlytics/CHANGELOG.md Outdated
# v4.0.0-beta.3

- [fixed] Fixed symbol collisions with the legacy Fabric Crashlytics SDK and added a warning not to include both (#4753, #4755)
- [fixed] Fixed an import declaration when installing using CocoaPods with the `generate_multiple_pod_projects` flag set to true (#4786)

@cbonnie cbonnie Jan 31, 2020

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Please change to:

"Fixed an import declaration for installing Crashlytics using CocoaPods. Previously, the declaration caused a compile error when you installed with the generate_multiple_pods_project flag set to true"

I'm not sure whether importing <FirebaseCore/FIRApp.h> is the same as "installing Crashlytics" so please revise if necessary.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Revision makes sense!

Comment thread Crashlytics/CHANGELOG.md Outdated
# v4.0.0-beta.3

- [fixed] Fixed an import declaration when installing using CocoaPods with the `generate_multiple_pod_projects` flag set to true (#4786)
- [fixed] Fixed an import declaration for installing Crashlytics using CocoaPods. Previously, the declaration caused a compile error when you installed using CocoaPods with the `generate_multiple_pods_project` flag set to true (#4786)

@cbonnie cbonnie Jan 31, 2020

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Please get rid of "using CocoaPods" :P So:

Fixed an import declaration for installing Crashlytics using CocoaPods. Previously, the declaration caused a compile error when you installed with the generate_multiple_pods_project flag set to true (#4786)

@samedson
samedson merged commit 8917b70 into master Jan 31, 2020
@samedson
samedson deleted the samedson-patch-1 branch January 31, 2020 18:06
ryanwilson added a commit that referenced this pull request Jan 31, 2020
* Crashlytics fix import declaration of FIRApp.h (#4786)

* Update Crashlytics CHANGELOG for beta.3 out of band release (#4788)

Co-authored-by: Sam Edson <samedson@google.com>
@firebase firebase locked and limited conversation to collaborators Mar 2, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants