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

♻️ Migrate tuple to NextArbitrary #1691

Merged
merged 11 commits into from
Apr 5, 2021
Merged

♻️ Migrate tuple to NextArbitrary #1691

merged 11 commits into from
Apr 5, 2021

Conversation

dubzzz
Copy link
Owner

@dubzzz dubzzz commented Apr 5, 2021

In a nutshell

❌ New feature
❌ Fix an issue
❌ Documentation improvement
✔️ Other: migrate to next gen

(✔️: yes, ❌: no)

Potential impacts

No impact expected BUT as we impact the code of tuples, changes may occur in its behaviour (not expected).

⚠️ Impact how deep values produced by letrec are biased. See #1700

@codesandbox-ci
Copy link

codesandbox-ci bot commented Apr 5, 2021

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 71d106c:

Sandbox Source
Vanilla Configuration
fast-check-examples Configuration

@codecov
Copy link

codecov bot commented Apr 5, 2021

Codecov Report

Merging #1691 (71d106c) into main (e921c79) will increase coverage by 0.15%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1691      +/-   ##
==========================================
+ Coverage   97.01%   97.17%   +0.15%     
==========================================
  Files          98       98              
  Lines        3488     3504      +16     
  Branches      681      684       +3     
==========================================
+ Hits         3384     3405      +21     
+ Misses        100       95       -5     
  Partials        4        4              
Flag Coverage Δ
unit-tests 97.17% <100.00%> (+0.15%) ⬆️
unit-tests-10.x-Linux 97.17% <100.00%> (+0.18%) ⬆️
unit-tests-12.x-Linux 97.17% <100.00%> (+0.18%) ⬆️
unit-tests-14.x-Linux 97.14% <100.00%> (+0.12%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/check/arbitrary/TupleArbitrary.ts 98.33% <100.00%> (+9.69%) ⬆️
src/check/symbols.ts 100.00% <0.00%> (+20.00%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e921c79...71d106c. Read the comment docs.

@dubzzz dubzzz changed the title Migrate tuple tonext ♻️ Migrate tuple to NextArbitrary Apr 5, 2021
@dubzzz
Copy link
Owner Author

dubzzz commented Apr 5, 2021

@github-actions deploy

@github-actions
Copy link
Contributor

github-actions bot commented Apr 5, 2021

Give a try to e5d01db with:

yarn add https://606b0b1a407c81229cea1d51--dazzling-goodall-4f0e38.netlify.app/fast-check.tgz
npm i https://606b0b1a407c81229cea1d51--dazzling-goodall-4f0e38.netlify.app/fast-check.tgz

Or have a look to the generated documentation.

@dubzzz
Copy link
Owner Author

dubzzz commented Apr 5, 2021

@github-actions deploy

@github-actions
Copy link
Contributor

github-actions bot commented Apr 5, 2021

Give a try to 3715461 with:

yarn add https://606b17b03520334ef18e0e4b--dazzling-goodall-4f0e38.netlify.app/fast-check.tgz
npm i https://606b17b03520334ef18e0e4b--dazzling-goodall-4f0e38.netlify.app/fast-check.tgz

Or have a look to the generated documentation.

@dubzzz dubzzz merged commit 6885298 into main Apr 5, 2021
@dubzzz dubzzz deleted the migrate-tuple-tonext branch April 5, 2021 15:02
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.

1 participant