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

Use reusable GitHub workflows for archiving and creating the XCFramework #9

Merged
merged 15 commits into from
Nov 10, 2023

Conversation

philippzagar
Copy link

@philippzagar philippzagar commented Nov 8, 2023

Use template GitHub workflows for archiving and creating the XCFramework

♻️ Current situation & Problem

In StanfordBDHG/.github#40, we introduced reusable GitHub Actions workflows for the archiving and creation of XCFrameworks. This enables us to avoid code duplication.

⚙️ Release Notes

  • GitHub Actions workflow utilizes shared reusable workflows.
  • ResearchKit updated to latest version v2.2.12

📚 Documentation

Provided in the workflow files

✅ Testing

Manually tested by creating an example release of ResearchKit.

Code of Conduct & Contributing Guidelines

By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines:

Pariecemckinney-apple and others added 8 commits November 1, 2023 18:18
- Fixed `ORKHealthKitQuantityTypeAnswerFormat`issues
- Improved `ORKReviewStep` initialization
- Introduced a new property called `shouldAutomaticallyAdjustImageTintColor` on `ORKStep` to automatically adopt dark mode version of an image
- Added `ORKSESQuestionResult` to Table View Providers
- ORKCatalog Improvements
- ORKImageSelectionView improvements
- Bumped version to 2.2.12
@philippzagar philippzagar self-assigned this Nov 9, 2023
@philippzagar philippzagar removed the request for review from PSchmiedmayer November 9, 2023 08:39
@philippzagar philippzagar force-pushed the feat/use-archive-and-xcframework-template-workflows branch from 3677a9f to 98dbd4e Compare November 9, 2023 08:41
@philippzagar
Copy link
Author

philippzagar commented Nov 9, 2023

@PSchmiedmayer Just created a new release via the reusable GitHub Actions workflows, worked fine: https://github.com/StanfordBDHG/ResearchKit/releases/tag/2.2.19

Feel free to check the (minimal) code changes (most of the diff comes from upstream ResearchKit) 🚀

Copy link
Member

@PSchmiedmayer PSchmiedmayer left a comment

Choose a reason for hiding this comment

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

Thank you for all the improvements. I had a few minor comments that I noted in the review, would be great if those are addressed before we merge the PR.

Thanks for all the work here to make this reusable 🚀

No need for an other review from my side once these issues are addressed and you validated that the feature branch is working using an external package and tests 👍

.github/workflows/build.yml Show resolved Hide resolved
.github/workflows/build.yml Show resolved Hide resolved
.github/workflows/build.yml Show resolved Hide resolved
.github/workflows/release.yml Show resolved Hide resolved
.github/workflows/build.yml Show resolved Hide resolved
ResearchKit.xcframework/Info.plist Show resolved Hide resolved
@philippzagar philippzagar merged commit 23ce5f2 into main Nov 10, 2023
2 checks passed
@philippzagar philippzagar deleted the feat/use-archive-and-xcframework-template-workflows branch November 10, 2023 17:02
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