-
Notifications
You must be signed in to change notification settings - Fork 106
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
Move Map Libre API key to Xcode Cloud #1313
Conversation
Generated by 🚫 Danger Swift against 0bd018d |
50cbc71
to
6bda3e8
Compare
3a1d7d5
to
8c88395
Compare
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.
LGTM
UnitTests/Sources/NotificationManager/NotificationManagerTests.swift
Outdated
Show resolved
Hide resolved
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.
Yeah I like this route, feels both cleaner and safer 🙏
2630a82
to
73c8ad8
Compare
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## develop #1313 +/- ##
===========================================
+ Coverage 44.16% 44.17% +0.01%
===========================================
Files 395 395
Lines 26023 26026 +3
Branches 13258 13259 +1
===========================================
+ Hits 11494 11498 +4
Misses 14236 14236
+ Partials 293 292 -1
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
32ee27f
to
0bd018d
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
This PR moves the map libre api key into Xcode Cloud (as a secret).
Xcode Cloud workflows will inject the key back into the app when creating a new production/nightly build.
It also adds a step in the
setup-project
action to configure the api key locally.