Skip to content

feat(napi): implement ValidateNapiValue for HashMap with any hasher#2374

Merged
Brooooooklyn merged 1 commit intonapi-rs:mainfrom
sapphi-red:feat/implement-validate-napi-value-for-hashmap-with-any-hasher
Nov 29, 2024
Merged

feat(napi): implement ValidateNapiValue for HashMap with any hasher#2374
Brooooooklyn merged 1 commit intonapi-rs:mainfrom
sapphi-red:feat/implement-validate-napi-value-for-hashmap-with-any-hasher

Conversation

@sapphi-red
Copy link
Contributor

For example, FromNapiValue and ToNapiValue was implemented for HashMap<K, V, BuildHasherDefault<FxHasher>>, but ValidateNapiValue was not.

@sapphi-red
Copy link
Contributor Author

The lint errors are not related to this PR.

@Brooooooklyn Brooooooklyn merged commit f116eaf into napi-rs:main Nov 29, 2024
@sapphi-red sapphi-red deleted the feat/implement-validate-napi-value-for-hashmap-with-any-hasher branch November 29, 2024 14:55
github-merge-queue bot pushed a commit to rolldown/rolldown that referenced this pull request Dec 2, 2024
<!-- Thank you for contributing! -->

### Description

This PR needs napi-rs/napi-rs#2374.

<!-- Please insert your description here and provide especially info
about the "what" this PR is solving -->
underfin pushed a commit to rolldown/rolldown that referenced this pull request Dec 9, 2024
<!-- Thank you for contributing! -->

### Description

This PR needs napi-rs/napi-rs#2374.

<!-- Please insert your description here and provide especially info
about the "what" this PR is solving -->
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