Skip to content
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

Add serde_as compatible versions for duplicate key handling #563

Merged
merged 3 commits into from
Mar 4, 2023

Conversation

jonasbb
Copy link
Owner

@jonasbb jonasbb commented Mar 4, 2023

This adds serde_as compatible versions for these four modules:
maps_duplicate_key_is_error, maps_first_key_wins,
sets_duplicate_value_is_error, and sets_last_value_wins.

Missing:

  • Documentation

Closes #534

jonasbb added 3 commits March 4, 2023 17:34
This adds `serde_as` compatible versions for these four modules:
`maps_duplicate_key_is_error`, `maps_first_key_wins`,
`sets_duplicate_value_is_error`, and `sets_last_value_wins`.

Closes #534
@codecov
Copy link

codecov bot commented Mar 4, 2023

Codecov Report

Merging #563 (ad49c0c) into master (ed04314) will decrease coverage by 0.62%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #563      +/-   ##
==========================================
- Coverage   50.72%   50.10%   -0.62%     
==========================================
  Files          37       38       +1     
  Lines        2289     2317      +28     
==========================================
  Hits         1161     1161              
- Misses       1128     1156      +28     
Impacted Files Coverage Δ
serde_with/src/de/duplicates.rs 0.00% <0.00%> (ø)
serde_with/src/de/mod.rs 87.50% <ø> (ø)
serde_with/src/rust.rs 55.88% <ø> (ø)
serde_with/src/ser/mod.rs 71.42% <ø> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@jonasbb
Copy link
Owner Author

jonasbb commented Mar 4, 2023

bors r+

@bors
Copy link
Contributor

bors bot commented Mar 4, 2023

Build succeeded:

@bors bors bot merged commit b50b611 into master Mar 4, 2023
@bors bors bot deleted the duplicate-handling branch March 4, 2023 17:20
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.

Using serde_with::rust::sets_last_value_wins with serde_as notation
1 participant