Skip to content

Follow the current 5.3.x plugin template - #15

Merged
seb5g merged 1 commit into
mainfrom
chore/follow-5.3.x-template
Oct 27, 2026
Merged

seb5g merged 1 commit into
mainfrom
chore/follow-5.3.x-template

Conversation

@seb5g

@seb5g seb5g commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

Before: the plugin followed the template of the first 5.3.x release only (PR #14). Since then the template moved: AGENTS.md was extended, the classifiers are built at packaging time from requires-python and the supported_platforms of the plugins, and the build requires pymodaq_utils>=5.3.6. This plugin still declared Python 3.8 and fixed classifiers (3.8 to 3.11).

After: the files that the template defines are the template ones. AGENTS.md and .gitignore are copied from the 5.3.x branch of pymodaq_plugins_template; pyproject.toml has requires-python = ">=3.10", the build requirement pymodaq_utils>=5.3.6, and the classifiers moved to [tool.pymodaq]. The plugin name, description, dependencies and urls are untouched.

How: compared every file of the template (git archive of 5.3.x) with the repository. The workflows, CLAUDE.md, hatch_build.py and the acceptance test were already the same. Checked locally with PyMoDAQ 5.3.x: the editable install builds and the metadata gets the classifiers for Python 3.10 to 3.13, pytest gives 3 passed and 1 skipped (the move plugin test has nothing to run: this package has no actuator), and check_plugin reports no error. It still lists two leftover template TODO comments in daq_2Dviewer_OpenCV.py, which the camera rewrite in the next pull request removes. No camera involved here.

🤖 Generated with Claude Code

https://claude.ai/code/session_01Tdiso1u1ZhikSutj2Qs5Cg

…kaging, Python >= 3.10

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Tdiso1u1ZhikSutj2Qs5Cg
@seb5g
seb5g merged commit d926509 into main Oct 27, 2026
7 checks passed
@seb5g
seb5g deleted the chore/follow-5.3.x-template branch October 27, 2026 14:39
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