Skip to content

feat: derive serde::[De-]Serialize for alignment interface structs#155

Merged
sebschmi merged 1 commit intoalgbio:mainfrom
jeeeesper:serialize-range
Feb 24, 2026
Merged

feat: derive serde::[De-]Serialize for alignment interface structs#155
sebschmi merged 1 commit intoalgbio:mainfrom
jeeeesper:serialize-range

Conversation

@jeeeesper
Copy link
Copy Markdown
Collaborator

Just so I don't need a wrapper struct in my own code :D
I figure, why not add this, as serde is behind a feature gate anyways.

Copy link
Copy Markdown
Collaborator

@sebschmi sebschmi left a comment

Choose a reason for hiding this comment

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

Looks good!

@sebschmi
Copy link
Copy Markdown
Collaborator

Could you still run cargo update to fix the nightly lint errors if fixable?

@jeeeesper
Copy link
Copy Markdown
Collaborator Author

Not (easily) fixable, it's a problem in nightly: rust-lang/rust#152961

@sebschmi sebschmi merged commit 45abb07 into algbio:main Feb 24, 2026
14 of 18 checks passed
@sebschmi
Copy link
Copy Markdown
Collaborator

Thanks for checking!

@jeeeesper jeeeesper deleted the serialize-range branch March 5, 2026 08:17
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