Skip to content

ci: Include macOS and Windows in CI pipeline - #22

Merged
Ki-Seki merged 13 commits into
MemTensor:devfrom
Ki-Seki:mac-win-adapt
Jul 8, 2025
Merged

ci: Include macOS and Windows in CI pipeline#22
Ki-Seki merged 13 commits into
MemTensor:devfrom
Ki-Seki:mac-win-adapt

Conversation

@Ki-Seki

@Ki-Seki Ki-Seki commented Jul 8, 2025

Copy link
Copy Markdown
Member

Expand the CI pipeline to include macOS and Windows environments for testing alongside Ubuntu.

Fix #21 #26

Copilot AI review requested due to automatic review settings July 8, 2025 07:09

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR extends the CI pipeline to run tests on Windows and macOS in addition to Ubuntu, and simplifies the pytest invocation.

  • Added windows-latest and macos-latest to the os matrix
  • Removed explicit PYTHONPATH setting in the pytest command
Comments suppressed due to low confidence (1)

.github/workflows/python-tests.yml:56

  • Verify that this command is indented exactly two spaces under the run: | block (matching the YAML structure used for other steps) to ensure the CI workflow remains valid.
        poetry run pytest tests -vv

@Ki-Seki Ki-Seki linked an issue Jul 8, 2025 that may be closed by this pull request
@Ki-Seki
Ki-Seki force-pushed the mac-win-adapt branch 6 times, most recently from e436664 to caefa04 Compare July 8, 2025 14:32
@Ki-Seki
Ki-Seki merged commit 5376947 into MemTensor:dev Jul 8, 2025
tianxing02 pushed a commit to tianxing02/MemOS that referenced this pull request Feb 24, 2026
* ci: include mac and win

* fix: use pathlib for cross-platform path handling in cube creation test

* fix: improve file path handling in test_dump using os.path.join

* fix: use a small hf model instead

* fix: add LLMFactory mock to test MOS initialization with invalid user

* fix: remove warning about platform support

* fix:db close in windows for sqlitedb

* fix: fix ci

* fix: delete db after close and get permission

* ci: explicitly checks macos-14 and macos-15

* docs: add warning about macOS compatibility

---------

Co-authored-by: fridayL <lcy081099@gmail.com>
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.

build: support macOS and Windows platforms

3 participants