Skip to content

Remove the serde feature and instead just always require it #2737

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 21, 2025

Conversation

Firestar99
Copy link
Collaborator

@Firestar99 Firestar99 commented Jun 18, 2025

Requires #2736

Everything needs serde anyway

@Firestar99 Firestar99 changed the base branch from master to workspace-dependencies June 18, 2025 16:31
@Firestar99 Firestar99 marked this pull request as ready for review June 18, 2025 16:59
@Keavon Keavon changed the title remove feature serde, always require it instead Remove feature serde, always require it instead Jun 18, 2025
@Keavon Keavon force-pushed the remove-serde-feature branch from 94727cb to 7e034a8 Compare June 20, 2025 06:57
@Keavon Keavon force-pushed the workspace-dependencies branch from da125b8 to 0c0551f Compare June 20, 2025 06:57
@Keavon Keavon force-pushed the workspace-dependencies branch from 0c0551f to cb8a6c3 Compare June 21, 2025 23:11
Base automatically changed from workspace-dependencies to master June 21, 2025 23:26
@Keavon Keavon force-pushed the remove-serde-feature branch from 7e034a8 to 95fff0b Compare June 21, 2025 23:27
@Keavon Keavon changed the title Remove feature serde, always require it instead Remove serde feature and instead just always require it Jun 21, 2025
@Keavon Keavon changed the title Remove serde feature and instead just always require it Remove the serde feature and instead just always require it Jun 21, 2025
@Keavon Keavon merged commit e520c21 into master Jun 21, 2025
4 checks passed
@Keavon Keavon deleted the remove-serde-feature branch June 21, 2025 23:52
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.

3 participants