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

Change Array by copy #537

Merged
merged 15 commits into from
Dec 6, 2022
Merged

Change Array by copy #537

merged 15 commits into from
Dec 6, 2022

Conversation

petamoriken
Copy link
Owner

@petamoriken petamoriken marked this pull request as draft November 3, 2021 11:34
@codecov
Copy link

codecov bot commented Nov 3, 2021

Codecov Report

Base: 87.87% // Head: 88.11% // Increases project coverage by +0.24% 🎉

Coverage data is based on head (79a78ab) compared to base (f714553).
Patch coverage: 93.75% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #537      +/-   ##
==========================================
+ Coverage   87.87%   88.11%   +0.24%     
==========================================
  Files          12       12              
  Lines         742      774      +32     
  Branches      138      142       +4     
==========================================
+ Hits          652      682      +30     
- Misses         48       49       +1     
- Partials       42       43       +1     
Flag Coverage Δ
unittests 88.11% <93.75%> (+0.24%) ⬆️

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

Impacted Files Coverage Δ
src/Float16Array.mjs 89.41% <93.54%> (+0.29%) ⬆️
src/_util/messages.mjs 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@petamoriken petamoriken force-pushed the feature/change-array-by-copy branch 2 times, most recently from 4d124a4 to 8e5ce04 Compare November 11, 2021 05:19
@petamoriken petamoriken force-pushed the feature/change-array-by-copy branch 2 times, most recently from 1f96a85 to d4ca9a4 Compare December 14, 2021 02:11
@petamoriken petamoriken marked this pull request as ready for review April 4, 2022 17:37
@petamoriken petamoriken force-pushed the feature/change-array-by-copy branch 3 times, most recently from edf3bc3 to 121b54f Compare June 26, 2022 07:43
@petamoriken petamoriken force-pushed the feature/change-array-by-copy branch 2 times, most recently from 2aaa661 to 5fe649a Compare July 25, 2022 15:11
@petamoriken petamoriken merged commit 7d1f718 into master Dec 6, 2022
@petamoriken petamoriken deleted the feature/change-array-by-copy branch December 6, 2022 11:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant