-
Notifications
You must be signed in to change notification settings - Fork 3
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
Support Strict Concurrency #30
Conversation
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.
Looks great; thank you @Supereg!
Just saw that this PR isn't merged yet. Could you force merge this PR @PSchmiedmayer? I currently do not have the permission for that. Codecov sadly wasn't able to provide codecov results after two attempts. |
It seems like the main branch never collected any coverage reports? Also the last few PRs also didn't collect any coverage. Maybe it is a configuration issue? |
@PSchmiedmayer I updated the build_and_test action to also run on a push to main. This should ensure code coverage will work in the future. |
Sure; thanks for the reminder! I just merged the PR 👍 |
thank you 🚀 |
Support Strict Concurrency
♻️ Current situation & Problem
This PR enables strict concurrency for the ResearchKitSwiftUI targets. Swift 6 is now fully supported.
⚙️ Release Notes
📚 Documentation
--
✅ Testing
--
Code of Conduct & Contributing Guidelines
By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines: