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

Testflight expire date #163

Merged
merged 2 commits into from
May 10, 2024
Merged

Conversation

bjorkert
Copy link
Contributor

@bjorkert bjorkert commented May 9, 2024

Calculation of expiration date that handles xcode and browser/testflight.
Refactoring of Branch/Tag/sha storage, using the BuildDetails.plist instead of branch.txt
Resolves: #153

image
image

@MikePlante1
Copy link
Contributor

MikePlante1 commented May 9, 2024

Tried to test this, but GH failed to upload the IPA. I don't think that's related to this PR at all, seems to be from removing BGTaskSchedulerPermittedIdentifiers with this commit.

Failed run log

Re-testing after restoring that line now.

@MikePlante1
Copy link
Contributor

After applying #164 it built fine and seems to work.

TF expiration info

@marionbarker
Copy link
Contributor

Repeating comment in both PR #163 and PR #164 because both are required.

Success for Browser Build with PR 163 + 164 added to dev (including the merged PR 165).

First build failed because I inadvertently modified swiftpm/Package.resolved
Once I reverted that change, it was successful.

@marionbarker marionbarker self-requested a review May 9, 2024 20:21
@marionbarker marionbarker mentioned this pull request May 9, 2024
@MikePlante1 MikePlante1 self-requested a review May 9, 2024 20:24
@MikePlante1 MikePlante1 merged commit 4995ee0 into nightscout:dev May 10, 2024
@MikePlante1 MikePlante1 mentioned this pull request May 10, 2024
4 tasks
@bjorkert bjorkert deleted the testflight-expire-date branch May 10, 2024 04:42
bjornoleh added a commit that referenced this pull request May 10, 2024
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.

Expiration date for browser build
3 participants