Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Ibis] Don't include design name in namespace in IbisContainersToHW #7425

Merged
merged 1 commit into from
Aug 6, 2024

Conversation

mortbopet
Copy link
Contributor

The ibis.design op will be removed after the IbisContainersToHW pass, and there may be ibis.component's inside the design that have the same name as the design; we want that name to persist, and not be falsely considered a duplicate.

@mortbopet mortbopet force-pushed the dev/mpetersen/ibis_no_alias_design_name branch 3 times, most recently from 33e6798 to 79975b5 Compare August 2, 2024 12:26
Copy link
Contributor

@blakep-msft blakep-msft left a comment

Choose a reason for hiding this comment

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

Thanks for doing this.

Base automatically changed from dev/mpetersen/namespace_erase to main August 6, 2024 12:28
The ibis.design op will be removed after the IbisContainersToHW pass, and there may be ibis.component's inside the design that have the same name as the design; we want that name to persist, and not be falsely considered a duplicate.
@mortbopet mortbopet force-pushed the dev/mpetersen/ibis_no_alias_design_name branch from 79975b5 to 22ea3ee Compare August 6, 2024 12:30
@mortbopet mortbopet merged commit 0b93783 into main Aug 6, 2024
4 checks passed
@mortbopet mortbopet deleted the dev/mpetersen/ibis_no_alias_design_name branch August 6, 2024 15:34
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.

2 participants