Skip to content

ci: pin EoSim to a tag that exists - #95

Merged
srpatcha merged 2 commits into
embeddedos-org:masterfrom
Kartikey1306:fix/eosim-sanity-real-tag
Sep 1, 2026
Merged

srpatcha merged 2 commits into
embeddedos-org:masterfrom
Kartikey1306:fix/eosim-sanity-real-tag

Conversation

@Kartikey1306

@Kartikey1306 Kartikey1306 commented Sep 1, 2026 •

Copy link
Copy Markdown
Contributor

EoSim Sanity has never had a green run in this repository. It is a nightly workflow, so it does not gate pull requests — it is simply red every morning.

The cause

EOSIM_VERSION is "0.1.0", and embeddedos-org/EoSim has no such tag. Its tags run v1.0.0 through v3.0.1:

$ git clone --depth 1 --branch v0.1.0 .../EoSim.git /tmp/EoSim
fatal: Remote branch v0.1.0 not found in upstream origin

$ pip install -e /tmp/EoSim
ERROR: file:///tmp/EoSim does not appear to be a Python project:
neither 'setup.py' nor 'pyproject.toml' found

The second message is a red herring that makes this look like an EoSim packaging problem. The directory is empty because the clone failed — EoSim is a perfectly normal Python project, and v1.5.0 carries a pyproject.toml declaring name = "eosim" with a console entry point.

Pinned to v1.5.0, the release marked Latest.

Verified, not assumed

I ran exactly what the workflow runs:

git clone --depth 1 --branch v1.5.0 .../EoSim.git /tmp/EoSim
pip install /tmp/EoSim        -> rc=0
eosim --version               -> eosim, version 2.0.0
eosim list                    -> rc=0
eosim doctor                  -> rc=0

Those four are every EoSim command this workflow invokes.

One caveat worth recording

eosim list prints Available platforms (0) while discover_platforms() on the same directory returns 149 — a display bug inside EoSim, not something this workflow can fix. It exits 0, so the sanity steps pass either way. Flagging it rather than leaving it to surprise the next person.

@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@Kartikey1306

Copy link
Copy Markdown
Contributor Author

Stacked on #66. The red/queued checks were master's CI matrix, not this change — it only edits .github/workflows/eosim-sanity.yml:

  • Windows fails with ParserError: Missing expression after unary operator '--' — the test step uses backslash continuations under PowerShell, so pytest never starts.
  • macOS jobs sit queued forever — the matrix pins macos-13, a retired runner image.

#66 fixes both.

@Kartikey1306
Kartikey1306 force-pushed the fix/eosim-sanity-real-tag branch from f3c99ff to 224a1a0 Compare September 1, 2026 07:12
EoSim Sanity has never had a green run in this repository. EOSIM_VERSION is
"0.1.0" and embeddedos-org/EoSim has no such tag -- its tags run v1.0.0 through
v3.0.1 -- so every job dies at the clone:

    fatal: Remote branch v0.1.0 not found in upstream origin

and then, predictably, at the install:

    ERROR: file:///tmp/EoSim does not appear to be a Python project:
    neither 'setup.py' nor 'pyproject.toml' found

The second message is a red herring: the directory is empty because the clone
failed, not because EoSim is not a Python project. It is one -- v1.5.0 carries
a pyproject.toml declaring name = "eosim" with a console entry point.

Pinned to v1.5.0, the release marked Latest. Verified by doing exactly what the
workflow does:

    git clone --depth 1 --branch v1.5.0 .../EoSim.git /tmp/EoSim
    pip install /tmp/EoSim        -> rc=0
    eosim --version               -> eosim, version 2.0.0
    eosim list                    -> rc=0
    eosim doctor                  -> rc=0

Those four are every EoSim command this workflow invokes.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@Kartikey1306
Kartikey1306 force-pushed the fix/eosim-sanity-real-tag branch from 224a1a0 to de9df40 Compare September 1, 2026 09:39
srpatcha
srpatcha previously approved these changes Sep 1, 2026
@Kartikey1306
Kartikey1306 dismissed srpatcha’s stale review September 1, 2026 10:13

The merge-base changed after approval.

@srpatcha
srpatcha merged commit e7f693c into embeddedos-org:master Sep 1, 2026
16 of 25 checks passed
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.

3 participants