Skip to content

Use IndexMap for mounts and validate duplicates [Namespaces Part 2]#5062

Open
aasoni wants to merge 3 commits into
masterfrom
alessandro/validate-module-mount-names
Open

Use IndexMap for mounts and validate duplicates [Namespaces Part 2]#5062
aasoni wants to merge 3 commits into
masterfrom
alessandro/validate-module-mount-names

Conversation

@aasoni
Copy link
Copy Markdown
Contributor

@aasoni aasoni commented May 18, 2026

Description of Changes

  • Replace ModuleDef mounts type with IndexMap rather than Vec to explicitly require name uniqueness
  • Validate mount name uniqueness

API and ABI breaking changes

No

Expected complexity level and risk

2 - Fairly simple change to new (as of yet unused) module def type

Testing

  • Added a test to check that the duplicate name check is working correctly

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