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

dev sync: several fixes and updates #735

Merged
merged 63 commits into from Jun 1, 2023
Merged

dev sync: several fixes and updates #735

merged 63 commits into from Jun 1, 2023

Conversation

pateketrueke
Copy link
Member

@pateketrueke pateketrueke commented Jan 2, 2023

Work in progress

This PR would resolve several mistakes from bundles, from now on the published packages should be stable for ESM and for usage on Deno.

It also includes other small contributions as well minor corrections, we want to include more fixes and small features before the next release!

@codecov-commenter
Copy link

codecov-commenter commented Jan 2, 2023

Codecov Report

Merging #735 (5fcfacc) into master (c79ea9a) will increase coverage by 1.36%.
The diff coverage is 35.71%.

❗ Current head 5fcfacc differs from pull request most recent head 362803e. Consider uploading reports for the commit 362803e to get more accurate results

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@            Coverage Diff             @@
##           master     #735      +/-   ##
==========================================
+ Coverage   61.33%   62.69%   +1.36%     
==========================================
  Files          37       37              
  Lines        2661     2694      +33     
  Branches      320      330      +10     
==========================================
+ Hits         1632     1689      +57     
+ Misses       1025      999      -26     
- Partials        4        6       +2     
Impacted Files Coverage Δ
src/lib/core/random.mjs 62.89% <0.00%> (+3.17%) ⬆️
src/lib/index.mjs 62.56% <0.00%> (ø)
src/lib/types/array.mjs 38.88% <0.00%> (-0.63%) ⬇️
src/lib/types/object.mjs 50.96% <0.00%> (+9.35%) ⬆️
src/lib/types/string.mjs 31.16% <0.00%> (-0.42%) ⬇️
src/lib/core/traverse.mjs 56.18% <50.00%> (+0.97%) ⬆️
src/lib/core/utils.mjs 64.68% <62.50%> (+0.04%) ⬆️
src/lib/api/defaults.mjs 100.00% <100.00%> (ø)
src/lib/generators/coreFormat.mjs 89.79% <100.00%> (+0.43%) ⬆️
src/lib/main.mjs 100.00% <100.00%> (ø)
... and 1 more

... and 1 file with indirect coverage changes

pateketrueke and others added 21 commits March 2, 2023 14:12
Allow empty arrays without defined items
fix:Option alwaysFakeOptionals to generate the wrong type in > 0.5.0-…
fix: when limit === 0, random number attributes should not be generat…
fix: exports types to fix TS7016 type error
fix:Option fillProperties set false, should not generate ambiguous at…
fix: alwaysFakeOptionals causes additionalProperties to generate the …
fix: object type should generate the correct data, should obtain the …
ci: code refactor written in dot notation
…red-fix

fix: use alwaysFakeOptionals, not required object type should be corr…
…keOptionals value for ci and add alwaysFakeOptionals true unit test.
pateketrueke and others added 5 commits April 21, 2023 15:11
ci: utils.shouldClean and utils.clean before test,should set alwaysFa…
…ot-rely-on-period-dot-character-for-timezone

fix: clampDateTime to not rely on `.` for timezone
@jonesphi
Copy link

@pateketrueke Do you have a target date for this release?

@pateketrueke pateketrueke merged commit 8709a76 into master Jun 1, 2023
2 checks passed
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.

None yet

10 participants