Skip to content

Declare the platforms supported by the instrument plugins - #2

Merged
seb5g merged 1 commit into
mainfrom
feature/supported-platforms
Oct 30, 2026
Merged

seb5g merged 1 commit into
mainfrom
feature/supported-platforms

Conversation

@seb5g

@seb5g seb5g commented Oct 30, 2026

Copy link
Copy Markdown
Contributor

Before:
Nothing told PyMoDAQ that the GoIO viewer only works on Windows (it uses the GoIO Windows DLL through a 32-bit server).

After:
DAQ_0DViewer_GoIO declares supported_platforms = ('win32',): PyMoDAQ refuses to initialize it elsewhere, with a message.

How: one class attribute (PyMoDAQ >= 5.3.5 reads it, older versions ignore it). This repository has no pyproject.toml on main yet, so no classifier is touched here: the packaging migration PR computes them from this declaration.

Tested: file compiles. Not tested: initialization on Windows or elsewhere with a real sensor; the declaration comes from the code reading.


Generated by Claude Code

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Tdiso1u1ZhikSutj2Qs5Cg

seb5g commented Oct 30, 2026

Copy link
Copy Markdown
Contributor Author

call_workflow / build is the legacy workflow of main (actions/checkout@v3, setup-python@v4, setup.py based) and exits with code 1 independently of this change, which only adds a class attribute. The packaging migration and the shared workflows are in the rollout PR #1, still open; I did not widen this PR.


Generated by Claude Code

@seb5g
seb5g merged commit ebd4ab8 into main Oct 30, 2026
1 check failed
@seb5g
seb5g deleted the feature/supported-platforms branch October 30, 2026 13:07
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.

2 participants