Skip to content

align the cmem-cmemc constraint with the template - #14

Merged
seebi merged 1 commit into
mainfrom
feature/alignCmemcConstraint
Sep 6, 2026
Merged

seebi merged 1 commit into
mainfrom
feature/alignCmemcConstraint

Conversation

@seebi

@seebi seebi commented Sep 6, 2026

Copy link
Copy Markdown
Member

Aligns the cmem-cmemc constraint with the template's own >=24.2.0.

The template shipped ^25.3.0 for a single release (v7.3.0) before lowering it again in v8.0.0 "to avoid a dependency deadlock". Repositories rendered in that window kept the caret, and copier's three-way merge treats it as project content — so updating all the way to v9.6.1 never restored the template value.

The caret caps cmem-cmemc below 26. That matters because click 8.5 now rejects an empty string as a colour and cmemc 25 passes one straight to click.secho, so any test driving cmemc through CliRunner fails with Unknown color '' — which is what broke cmem-plugin-shapes and cmem-plugin-wfreports.

This repository does not invoke cmemc in its tests, so this is alignment, not a fix. Its only cmemc use is task install / task uninstall.

The template briefly shipped ^25.3.0 (v7.3.0 only) before lowering it back to
>=24.2.0 in v8.0.0, to avoid a dependency deadlock. Repositories rendered in
that window kept the caret, and copier's three-way merge treats it as project
content, so updating all the way to v9.6.1 never restored the template value.

The caret caps cmem-cmemc below 26, which matters now that click 8.5 rejects an
empty colour and cmemc 25 passes one through: any test driving cmemc through
click's CliRunner fails with `Unknown color ''`. This repository does not do
that today, so this is alignment rather than a fix.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01UhUJZ7EwjXm7mN2jAdZzvn
@github-actions

github-actions Bot commented Sep 6, 2026

Copy link
Copy Markdown

Coverage

Coverage Report
File Stmts Miss Cover Missing
init.py 0 0 100%
plugin_pyshacl.py 253 7 97% 86 88-89 92-93 487 493
TOTAL 253 7 97%  

Tests Skipped Failures Errors Time
22 0 💤 0 ❌ 0 🔥 79.479 ⏱

@seebi
seebi merged commit 4f37a34 into main Sep 6, 2026
2 checks passed
@seebi
seebi deleted the feature/alignCmemcConstraint branch September 6, 2026 19:47
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.

1 participant