Skip to content

fix(ci): download wheels from artifacts#121

Open
Anillc wants to merge 5 commits into
openecos-projects:mainfrom
Anillc:patch-4
Open

fix(ci): download wheels from artifacts#121
Anillc wants to merge 5 commits into
openecos-projects:mainfrom
Anillc:patch-4

Conversation

@Anillc

@Anillc Anillc commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Summary

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 - Bazel, Nix, PyInstaller, wheels, uv.lock, or release artifacts.
  • CI/release - GitHub Actions, version checks, changelog, or release automation.
  • Tests/docs only

What Changed

Validation

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

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

Skipped checks and reason:

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, Bazel, or release artifact changed

Notes:

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.

@Anillc Anillc force-pushed the patch-4 branch 2 times, most recently from 6594bbc to b47ee3b Compare June 22, 2026 04:08
@Anillc Anillc marked this pull request as draft June 22, 2026 05:30
@Anillc Anillc force-pushed the patch-4 branch 8 times, most recently from 9b12116 to a4c59a6 Compare June 22, 2026 08:49
@Anillc Anillc marked this pull request as ready for review June 23, 2026 08:21
Co-authored-by: Emin <me@emin.chat>
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