-
-
Notifications
You must be signed in to change notification settings - Fork 872
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
Define SDK compatibility #1755
Comments
Thanks for opening this issue!
|
This wouldn’t work as GitHub Actions only supports 5 concurrent jobs on MacOS images for the entire repo. A build matrix you want would take hours if not days to run. Not to mention actually implementing it.
Github Badges for supported version have been added to the README, this should be enough.
This is the current policy, no need to document it as it is standard everywhere.
|
The run time length is something we can solve with more runners. What's the challenge in the implementation? Is it more than just adding a matrix to the CI workflow like we do with Node.js for example?
I wouldn't know how to interpret these badges. For example, what does the
We could still have a different policy, a developer currently can only guess what our policy is, so we need to define it. We do that in all other Parse SDKs. |
New Feature / Enhancement Checklist
Current Limitation
This Parse SDK is lacking a proper OS compatibility policy and CI tests.
Feature / Enhancement Description
Add CI jobs which test against the major OS versions for which this Parse SDK is claiming compatibility. For example, if it supports watchOS, the tests needs to run in watchOS 2, 3, 4, ... up to the highest version it is claiming to be compatible with.
Accordingly, a compatibility table needs to be added to the README that indicated the compatible - and tested - OS versions.
The SDK also lacks a OS support policy that defines when support for older OS versions is dropped. For example, watchOS 2 has been released 8 years ago, so its support should likely be dropped. Currently, the min supported version should likely be watchOS 9 because watchOS 8 has reached its EOL and doesn't receive security updates anymore. The only source for the support dates I found was a 3rd party website, but we should base the support policy on an official source from Apple.
The text was updated successfully, but these errors were encountered: