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

Temp PR #2

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

tyshko5
Copy link
Owner

@tyshko5 tyshko5 commented Nov 22, 2022

No description provided.

fix: make the deserializer human-unreadable (ipld#165)

Currently, the Serializer sets `is_human_readable` to false, but the Deserializer implicitly sets it to true. This causes issues when other types try to use serde based on this value, as in zcash/pasta_curves#48, so this PR makes them consistently false.

BREAKING CHANGE: the Serde deserializer is now defined as non human readable

lint fix

try fix CI

empty

revert

try fix 2

fix ci 2

fix ci 3

less branches no dep:, arb feature

fix arb

lint fixes

fix wasm ci

fix arb 3

fix wasm ci

fix wasm ci 2

fix no_std build

requested changes

getrandom optional

remove getrandom

fix recursion

requested changes

shrink

fix duplicate code

update `multihash` in other crates

fix

(cargo-release) version 0.15.0

(cargo-release) version 0.15.0

(cargo-release) version 0.15.0

(cargo-release) version 0.15.0

(cargo-release) version 0.15.0

(cargo-release) version 0.15.0

(cargo-release) version 0.15.0

fixes

minor

remove custom generator

remove arbitrary_ipld_vec/map fn_s

remove filtering of fleat nans

make arbitrary_ipld private

copy size to avoid too small lists/maps

fix arbitrary_size with actual size

fix size == 0 edgecase

fmt && clippy

cliipy

chore: fix Android build on CI (ipld#169)

Also cache `cargo-apk` for faster CI run times.

move everything to arb module

fmt
@tyshko5 tyshko5 force-pushed the tyshko5/add-quickcheck-arbitrary-for-ipld branch from 2b7874d to 7d85b03 Compare November 29, 2022 16: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.

2 participants