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: livedemostore appetize builds #731

Merged
merged 52 commits into from
Nov 13, 2023
Merged

Conversation

NQuinn27
Copy link
Contributor

@NQuinn27 NQuinn27 commented Nov 8, 2023

This PR adds:

  • publishing of livedemostore links for our Appetize builds.
  • adding the livedemostore urls to our Slack notifications
  • Refactors our appetize build and upload action to a composite action for reuse.

Three kinds of builds are sent to Appetize:

Preview
A preview build is published to Appetize for every PR. The livedemostore link will look like this: https://livedemostore.common.primer.io/ios/preview_731 - where the number at the end corresponds to the PR number.
Note that there will only be one build active per PR. The build is overwritten on each subsequent build - I.E the the only build available for a PR is the most recent one.

Nightly
A nightly build is is published to appetize each night along with our UI tests. The livedemostore URL looks like this: https://livedemostore.common.primer.io/ios/nightly_10.11.2023 - where the date of the build follows the nightly_ portion.

Release
A release build is published to appetize along with our post-release PR merge action. The livedemostore url looks like this: https://livedemostore.common.primer.io/ios/release_2.18.0 - where the version number corresponding with the iOS SDK version is the final portion.

Screenshot 2023-11-11 at 10 47 10

Copy link
Contributor

github-actions bot commented Nov 8, 2023

Warnings
⚠️ Please assign someone aside from CODEOWNERS (@checkout-pci-reviewers) to review this PR.

Generated by 🚫 Danger Swift against b17c1a5

Copy link
Contributor

github-actions bot commented Nov 8, 2023

Appetize link:

@@ -11,7 +11,7 @@ import XCTest

class PrimerBancontactCardDataManagerTests: XCTestCase {

private static let expectationTimeout = 1.0
private static let expectationTimeout = 5.0
Copy link
Contributor Author

Choose a reason for hiding this comment

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

These were failing all through Friday so I bumped up the timeout, lets see how they do now. cc @jnewc

Copy link
Contributor Author

@NQuinn27 NQuinn27 Nov 11, 2023

Choose a reason for hiding this comment

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

Seems it was Xcode 15 that was flaky. Reverted to Xcode 14 for Appetize build + upload (as it was before), and have a note to discuss in next chapter meeting

@NQuinn27 NQuinn27 marked this pull request as ready for review November 11, 2023 09:48
@NQuinn27 NQuinn27 requested a review from a team as a code owner November 11, 2023 09:48
Copy link
Contributor

@borisprimer borisprimer left a comment

Choose a reason for hiding this comment

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

great work!

.github/actions/appetize-build/action.yml Outdated Show resolved Hide resolved
.github/workflows/build_test_upload.yml Outdated Show resolved Hide resolved
.github/workflows/build_test_upload.yml Outdated Show resolved Hide resolved
.github/workflows/post-release-merge.yml Outdated Show resolved Hide resolved
@NQuinn27 NQuinn27 merged commit 3027558 into master Nov 13, 2023
5 checks passed
@NQuinn27 NQuinn27 deleted the nq/livedemostore_appetize_builds branch November 13, 2023 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants