Skip to content

Add comprehensive trait implementations and testing#162

Open
RenaudDenis wants to merge 1 commit intoBoddlnagg:masterfrom
cutoff:feature/common-traits
Open

Add comprehensive trait implementations and testing#162
RenaudDenis wants to merge 1 commit intoBoddlnagg:masterfrom
cutoff:feature/common-traits

Conversation

@RenaudDenis
Copy link
Contributor

Add essential trait implementations (Debug, Hash, PartialEq, Eq) to core MIDI types for better production usage and testability.

The changes improve:

  • Port usage as hash map keys
  • Testing capabilities
  • Error messages via Debug implementations

Breaking changes: None

@RenaudDenis RenaudDenis marked this pull request as draft November 28, 2024 13:56
@RenaudDenis RenaudDenis force-pushed the feature/common-traits branch 2 times, most recently from 81768fa to b1783fd Compare November 28, 2024 18:05
Add essential trait implementations (Debug, Hash, PartialEq, Eq) to core MIDI types for better production usage and testability.

The changes improve:
- Port usage as hash map keys
- Testing capabilities
- Error messages via Debug implementations

Breaking changes: None
@RenaudDenis RenaudDenis force-pushed the feature/common-traits branch from da78c4e to c0a253b Compare November 28, 2024 18:16
@RenaudDenis RenaudDenis marked this pull request as ready for review November 28, 2024 18:20
@RenaudDenis
Copy link
Contributor Author

Hi @Boddlnagg, nice to meet you.
Painful trial and errors to make it work on all platforms. I would welcome any advice to ease the process 😅
I hope you will find this to be a useful addition to the crate.

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