Skip to content

Strict params - #552

Draft
ducky64 wants to merge 21 commits into
masterfrom
strict-params
Draft

ducky64 wants to merge 21 commits into
masterfrom
strict-params

Conversation

@ducky64

@ducky64 ducky64 commented Sep 27, 2026 •

Copy link
Copy Markdown
Collaborator

Enforces all params (including in ports, excluding arg-params) in non-abstract blocks have assigned values.

Incidental refactors:

  • Footprint params are now assigned
  • Refactors core tests which ignored unassigned params
  • Adds Block._is_abstract internal API
  • Fix param binding type in packed blocks

TODOs:

  • Change to deprecation for now
  • Refactor initializers to generate assigns
  • Enforce absence of params for connected ports
  • Multipack _packed_connected_ports deduplication
  • DRYify _check_params_assigned
  • Assert multipacked inner assignments

Follow-on PRs:

  • Refactor to use Block.is_abstract API

Contributor License Agreement

By submitting this pull request, I agree to dual-license this contribution under both BSD 3-clause and Apache License, Version 2.0.

  • I agree to dual-license this contribution under both BSD 3-clause and Apache License, Version 2.0.

This branch has not been deployed

No deployments
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