Skip to content

feat: Add Display Implementation for NodeIndex<Ix> and EdgeIndex<Ix> whenever Ix: Display.#941

Open
RaoulLuque wants to merge 2 commits into0.8from
rl/impl_display_for_nodeindex
Open

feat: Add Display Implementation for NodeIndex<Ix> and EdgeIndex<Ix> whenever Ix: Display.#941
RaoulLuque wants to merge 2 commits into0.8from
rl/impl_display_for_nodeindex

Conversation

@RaoulLuque
Copy link
Member

@RaoulLuque RaoulLuque commented Jan 8, 2026

This PR implements the Display trait for NodeIndex<Ix> and EdgeIndex<Ix>, if Ix: Display.

During the implementation of the new Traits on the "old" graph types as described in #891, we noticed that these Impls would be useful.

and EdgeIndex<Ix> where Ix: Display.
@RaoulLuque RaoulLuque force-pushed the rl/impl_display_for_nodeindex branch from 094c166 to bdaa579 Compare January 8, 2026 16:21
@RaoulLuque RaoulLuque changed the title feat: Add Display Implementation for NodeIndex<Ix> where Ix: Display. feat: Add Display Implementation for NodeIndex<Ix> and EdgeIndex<Ix> where Ix: Display. Jan 8, 2026
@RaoulLuque
Copy link
Member Author

CI seems to be failing unrelated to this. I guess that's another TODO

@RaoulLuque RaoulLuque force-pushed the rl/impl_display_for_nodeindex branch from a531cc8 to 2277cdb Compare January 17, 2026 14:25
@RaoulLuque RaoulLuque changed the title feat: Add Display Implementation for NodeIndex<Ix> and EdgeIndex<Ix> where Ix: Display. feat: Add Display Implementation for NodeIndex<Ix> and EdgeIndex<Ix> whenever Ix: Display. Jan 18, 2026
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