Skip to content

feat(report,ci): batch CSV export and FileCheck metrics product - #287

Open
plitenh wants to merge 3 commits into
openecos-projects:mainfrom
plitenh:test/signoff-lit-artifacts
Open

plitenh wants to merge 3 commits into
openecos-projects:mainfrom
plitenh:test/signoff-lit-artifacts

Conversation

@plitenh

@plitenh plitenh commented Sep 15, 2026

Copy link
Copy Markdown

What Changed

Scope

Select the areas touched by this PR:

  • CLI - command behavior, Typer command surface, output formats, or workspace commands.
  • Flow/runtime - workspace lifecycle, EngineFlow, step execution, logs, metrics, or artifacts.
  • EDA integration - Yosys, ECC-Tools, DreamPlace, KLayout, PDKs, or native/runtime wrappers.
  • Build/package - Nix, PyInstaller, wheels, uv.lock, or release artifacts.
  • CI/release - GitHub Actions, version checks, changelog, or release automation.
  • Tests/docs only

Runtime And Packaging Impact

  • No runtime or packaging impact
  • CLI output or machine-readable contract changed
  • Workspace layout, flow state, or artifact paths changed
  • Native toolchain or wrapper behavior changed
  • ecc-tools or ecc-dreamplace dependency changed
  • PyInstaller, Nix, or release artifact changed

Notes:

Validation

List the commands you ran. Mark checks that are not applicable as N/A.

  • uv run pytest test/
  • uv run ruff check chipcompiler test
  • uv run ruff format --check chipcompiler test
  • PyInstaller smoke: ecc --help, ecc --version, ecc version --json
  • Nix smoke: nix run .#cli -- --help
  • Manual flow smoke:
  • Other:

Skipped checks and reason:

Checklist

  • I kept the change scoped to ECC.
  • I updated docs or user-facing CLI text where behavior changed.
  • I included lockfile or version metadata updates when dependencies changed.
  • I documented any submodule updates and why they are needed.
  • I did not include local caches, virtual environments, or generated build outputs.
  • I explained skipped validation and remaining risk.

@plitenh

plitenh commented Sep 15, 2026

Copy link
Copy Markdown
Author

CI passed on pre-squash tip of this branch: https://github.com/plitenh/ecc/actions/runs/34922548247 (d7f26b6; current tip d28aad2 is a rebase/squash of the same change set)

Provide test/support CSV render/spec/projection for CI harnesses, with
unit coverage and an ics55_gcd FileCheck contract on present=true gates.
Split unit pytest from a packaged-ecc job that builds PyInstaller, runs
ecc init/run/signoff export, and FileChecks CSV projection by exit code.
Skip ICS55 filelist integration cases when the PDK is absent.
Expose nix run apps for ci-run-ics55-gcd, ci-export-signoff-csv, and
filecheck so versions are managed in nix/signoff-tools.nix instead of uv.
@plitenh
plitenh force-pushed the test/signoff-lit-artifacts branch from 9453a07 to f3ecbae Compare September 15, 2026 10:43
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