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

🦺[Tech Debt] Fixed podfile: Firebase cocoapod #78

Merged
merged 1 commit into from
Sep 2, 2021

Conversation

kcw-grunt
Copy link
Contributor

  • Cleaned up podfile
  • Turning off the code coverage allows the SwiftUI canvas to work

@kcw-grunt kcw-grunt requested a review from JCHeist July 11, 2021 21:07
pod 'Firebase/Analytics', '~> 6.0'
pod 'Firebase/Crashlytics', '~> 6.0'
# add pod 'SwiftLint'
pod 'Firebase/Analytics'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Bad edit from months ago. This updates the Firebase pod

@@ -28,7 +28,6 @@
selectedLauncherIdentifier = "Xcode.DebuggerFoundation.Launcher.LLDB"
shouldUseLaunchSchemeArgsEnv = "YES"
enableASanStackUseAfterReturn = "YES"
codeCoverageEnabled = "YES"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is a Xcode bug. You can't run code coverage and run SwiftUI canvas

@JCHeist
Copy link
Contributor

JCHeist commented Jul 12, 2021

Looks good other than the test faking for some reason. Why is that?

@kcw-grunt
Copy link
Contributor Author

Looks good other than the test faking for some reason. Why is that?

Bitrise has a step that can run pod install or use the cache., based on the Podfile.lock. I need to take a look..

Copy link
Contributor

@JCHeist JCHeist left a comment

Choose a reason for hiding this comment

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

Reran test on Bitrise, Tests now passing, if this PR is still applicable, good to merge now.

@kcw-grunt kcw-grunt force-pushed the techdebt/fix-podfile-firebase-76 branch from 52715a7 to 6872169 Compare September 2, 2021 15:07
- Cleaned up podfile
- Turning off the code coverage allows the SwiftUI canvas to work
@kcw-grunt kcw-grunt force-pushed the techdebt/fix-podfile-firebase-76 branch from 6872169 to 6606766 Compare September 2, 2021 15:10
@kcw-grunt kcw-grunt merged commit 3d653e9 into develop Sep 2, 2021
@kcw-grunt kcw-grunt deleted the techdebt/fix-podfile-firebase-76 branch September 2, 2021 15:51
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.

2 participants