Skip to content

Rename Dep marker to Dependency and align published wiki with copier template#244

Merged
timkpaine merged 2 commits into
mainfrom
tkp/dependency-marker-wiki
Jul 10, 2026
Merged

Rename Dep marker to Dependency and align published wiki with copier template#244
timkpaine merged 2 commits into
mainfrom
tkp/dependency-marker-wiki

Conversation

@timkpaine

@timkpaine timkpaine commented Jul 10, 2026

Copy link
Copy Markdown
Member

Follow-up to #239 (now merged). Two independent changes:

1. Rename the Dep marker to Dependency

The explicit container-dependency marker added in #239 was named Dep. Every other public dependency-related type in the library spells the word out (DependencySpec, EvaluationDependency), as do the sibling markers FromContext and Lazy. This renames the public marker DepDependency for consistency:

from ccflow import Dependency, Flow, FromContext

@Flow.model
def total(values: list[Dependency[int]]) -> int:
    return sum(values)

Also silences a benign Pydantic serializer warning: model_dump_json() on a Dependency-marked model containing a CallableModel leaf previously emitted PydanticSerializationUnexpectedValue. The generated (schema-only, SkipValidation) field annotation for dependency-slot params now maps to Any, so no warning is emitted; ccflow still validates leaves against the real annotation.

2. Fix the published wiki (align with copier template)

The GitHub wiki had duplicate/colliding Contribute, Build-from-Source, and Local-Development-Setup pages. The copier template (python-project-templates/base) stores these under docs/wiki/contribute/, and the copier uplift (bd35d2e) added them there but left the old top-level copies behind. GitHub's wiki engine flattens page slugs, so the two sets collided.

This removes the stale top-level duplicates and keeps the contribute/ versions, which are byte-identical to the template's rendered output. The flat sidebar links ([Build from Source](Build-from-Source)) continue to resolve to the contribute/ pages via gollum.

@github-actions

github-actions Bot commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Test Results

    1 files  ±0      1 suites  ±0   2m 42s ⏱️ -5s
1 209 tests ±0  1 207 ✅ ±0  2 💤 ±0  0 ❌ ±0 
1 215 runs  ±0  1 213 ✅ ±0  2 💤 ±0  0 ❌ ±0 

Results for commit 31763ea. ± Comparison against base commit 4a18b4e.

♻️ This comment has been updated with latest results.

@codecov

codecov Bot commented Jul 10, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 90.38462% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 93.02%. Comparing base (4a18b4e) to head (31763ea).

Files with missing lines Patch % Lines
ccflow/_flow_model_binding.py 75.00% 2 Missing and 1 partial ⚠️
ccflow/examples/flow_model/flow_model_example.py 0.00% 1 Missing ⚠️
...amples/flow_model/flow_model_hydra_builder_demo.py 0.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #244   +/-   ##
=======================================
  Coverage   93.02%   93.02%           
=======================================
  Files         164      164           
  Lines       18610    18612    +2     
  Branches     1234     1235    +1     
=======================================
+ Hits        17312    17314    +2     
  Misses       1068     1068           
  Partials      230      230           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Signed-off-by: Tim Paine <3105306+timkpaine@users.noreply.github.com>
…plicates

Signed-off-by: Tim Paine <3105306+timkpaine@users.noreply.github.com>
@timkpaine timkpaine force-pushed the tkp/dependency-marker-wiki branch from ee901bb to 31763ea Compare July 10, 2026 18:58
@timkpaine timkpaine merged commit c75263f into main Jul 10, 2026
19 of 20 checks passed
@timkpaine timkpaine deleted the tkp/dependency-marker-wiki branch July 10, 2026 19:13
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