-
Notifications
You must be signed in to change notification settings - Fork 321
Open
Labels
Description
NOTE: this issue predates this project's rename to Volta.
As @mikrostew pointed out in #221, it's confusing to have foo::Bar and foo::serial::Bar types have the same name, especially with foo::serial::Bar::into_bar().
Example: notion_core::inventory::NodeIndex and notion_core::inventory::serial::NodeIndex.
Some possible naming conventions:
notion_core::inventory::serial::NodeIndexAst(i.e., AST)notion_core::inventory::serial::NodeIndexIr(i.e., IR)notion_core::inventory::serial::NodeIndexSerialnotion_core::inventory::serial::NodeIndexSerdenotion_core::inventory::serial::NodeIndexSDnotion_core::inventory::serial::SerialNodeIndexnotion_core::inventory::serial::SerdeNodeIndexnotion_core::inventory::serial::SDNodeIndex
I don't love any of these. Other ideas?