Expand CI pipeline to support Windows and macOS
#160
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jimboid
approved these changes
Sep 17, 2025
There was a problem hiding this comment.
I would avoid where possible using -latest in CI. They tend to migrate latest quite early which has in the past led to breaking issues in software stacks. I'm approving here though since a python project is unlikely to suffer this unless the early releases of a new OS have limited python versions available at first release of the env. I've mostly seen this in compilation based and containerised applications, but in general you should version them to control what is available.
Other than this point, great addition to CI.
harryswift01
force-pushed
the
159-expand-ci-testing
branch
from
0942035 to
149dea3
Compare
|
Coveralls MacOS runner is currently not working, coverallsapp/github-action#248 |
- move the `setup()` and `teardown()` functions to a `test_base.py` file to centrally manage the setup and teardown of files - use `logging.shutdown()` to ensure logging handlers are correctly closed and shutdown after use to avoid file IO issues within tests - ensure the correct format of `jobxxx` folders are being used within tests ti ensure consistency
- BaseTestCase: improved docstrings, simplified setup/teardown, pre-create logs folder, restore working directory after tests, ensure temp dir cleanup. - TestEntropyManager: disable MDAnalysis and commands loggers to prevent logging noise during tests. - TestLoggingConfig: use self.test_dir consistently instead of temp_dir.name for log file paths. - TestMain: force UTF-8 mode for subprocess (-X utf8) and capture output with UTF-8 encoding to avoid UnicodeEncodeErrors on Windows. - Inlcude `windows2025` within `project-ci.yaml` as the test cases now work on Windows
…ss-platform path comparisons and proper handling of mixed, invalid, and non-job folder names
…ndows/Linux/macOS assertion failures
…CodeEntropy into 159-expand-ci-testing
- Removed the warning about Windows not being tested - Added quotation marks around the editable `pip install` command to ensure correct installation
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This PR expands the CI pipeline for
CodeEntropyto include cross-platform testing and multiple Python versions, improving coverage and reliability.Changes
Cross-Platform CI Testing:
ubuntu-24.04,windows-2025, andmacos-15.Impact
CodeEntropy's compatibility across major operating systems.