Skip to content

remove causal prefixes - #1501

Merged
lucasverleyen merged 1 commit into
masterfrom
issue1500_causalPrefixes
Aug 7, 2026
Merged

lucasverleyen merged 1 commit into
masterfrom
issue1500_causalPrefixes

Conversation

@lucasverleyen

@lucasverleyen lucasverleyen commented Aug 4, 2026

Copy link
Copy Markdown
Member

Fixes #1500
@qlambert-pro fyi
@LoneMeertens @annadellisola When unit tests pass, this PR is ready for a final check by you.

@annadellisola annadellisola left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @lucasverleyen @qlambert-pro for addressing this issue.
I do agree with removing the causal prefix from the bus declaration. The causality is already expressed through the block boundary signals, while the bus itself acts as a shared structured connector.

@LoneMeertens LoneMeertens left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Thanks for addressing this, @lucasverleyen and @qlambert-pro.

@lucasverleyen
lucasverleyen merged commit 0127753 into master Aug 7, 2026
2 checks passed
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.

Remove causal prefixes for expandable connectors

3 participants