You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently, CLP package launch issues (like the Python compatibility issue in #693) are only discovered after merging changes. Developers usually test on a single platform, such as Ubuntu Focal OR Ubuntu Jammy, and such compatibility issues are not easily discovered unless tests are performed on all platforms. Adding a package launch test to the PR check workflow would help catch these issues earlier in the development cycle.
Feature Request
Add a step to the PR check workflow that:
Builds the CLP package
Attempts to launch it in an environment similar to production
Alternatively, for the issue in #693 specifically, a linter rule can be added to disallow syntaxes that are only available in newer Python versions, if such rules exist.
Background
Currently, CLP package launch issues (like the Python compatibility issue in #693) are only discovered after merging changes. Developers usually test on a single platform, such as Ubuntu Focal OR Ubuntu Jammy, and such compatibility issues are not easily discovered unless tests are performed on all platforms. Adding a package launch test to the PR check workflow would help catch these issues earlier in the development cycle.
Feature Request
Add a step to the PR check workflow that:
This would help catch issues like:
References
initialize-results-cache.py
. #693/cc @junhaoliao
The text was updated successfully, but these errors were encountered: