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

ci: Fix TestFlight upload for previews #972

Merged
merged 7 commits into from
Mar 4, 2021

Conversation

philipphofmann
Copy link
Member

@philipphofmann philipphofmann commented Mar 3, 2021

📜 Description

App Store connect requires a period-separated list of at most three non-negative integers
for the version. Therefore we remove now the preview version suffixes with Fastlane before
uploading.

💡 Motivation and Context

The upload of iOS-Swift to TestFlight failed.

💚 How did you test it?

With CI.

📝 Checklist

  • I reviewed the submitted code
  • I updated the CHANGELOG if needed
  • I updated the docs if needed
  • Review from the native team if needed
  • No breaking changes

🔮 Next steps

App Store connect requires a period-separated list of at most three non-negative integers
for the version. Therefore the version bump script must not add the suffixes of preview
releases. This is fixed now.
@philipphofmann philipphofmann requested a review from a team March 3, 2021 08:32
@philipphofmann philipphofmann enabled auto-merge (squash) March 3, 2021 08:43
Philipp Hofmann added 2 commits March 3, 2021 09:47
…y/sentry-cocoa into ci/fix-app-store-connect-upload
@philipphofmann philipphofmann disabled auto-merge March 3, 2021 09:00
@codecov-io
Copy link

codecov-io commented Mar 3, 2021

Codecov Report

Merging #972 (c16a433) into master (d89f3b0) will increase coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #972      +/-   ##
==========================================
+ Coverage   94.73%   94.76%   +0.02%     
==========================================
  Files          78       78              
  Lines        3589     3589              
==========================================
+ Hits         3400     3401       +1     
+ Misses        189      188       -1     
Impacted Files Coverage Δ
Sources/Sentry/SentryThreadInspector.m 96.29% <0.00%> (-3.71%) ⬇️
Sources/Sentry/SentryInstallation.m 95.45% <0.00%> (+9.09%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d89f3b0...c16a433. Read the comment docs.

Copy link
Member

@bruno-garcia bruno-garcia left a comment

Choose a reason for hiding this comment

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

Lgtm

@philipphofmann philipphofmann merged commit fa45ab0 into master Mar 4, 2021
@philipphofmann philipphofmann deleted the ci/fix-app-store-connect-upload branch March 4, 2021 08:12
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.

3 participants