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

v3.2.0 #178

Merged
merged 23 commits into from
Oct 14, 2024
Merged

v3.2.0 #178

merged 23 commits into from
Oct 14, 2024

Conversation

dhershman1
Copy link
Owner

@dhershman1 dhershman1 commented Oct 13, 2024

📄 Description

Adds some new functionality, adds back in coverage reporting, and moves unit tests to a *.spec.js format

🦑 Type of Change

  • Bug Fix
  • Chore
  • New Feature
  • Breaking Change

📋 PR Checklist

  • This pull request has a descriptive title and information useful to a reviewer.
  • All tests are passing
  • Changelog Updated Properly
  • Types updated/added as needed
  • Has new/updated tests

📝 Changes

🚨 Breaking Changes

  • List of breaking changes if any

💅 New

  • Added objOf function
    • Creates a new object based on the key value pair provided
  • fuzzySearch has been Deprecated as of this version and will be removed in v4.0.0

🎊 Enhanced

  • amend now uses Object spread rather than Object.assign for that small performance boost
    • When using Object.assign with an empty object, Object spread is consistently faster
  • Converted map to use a while loop assoc combo
  • Moved *.js -> *.spec.js for unit tests
  • Expanded upon some more tests to get to 100% coverage

🔧 Fixed

  • Fixed coverage reporting by switching to the c8 package

@dhershman1 dhershman1 added enhancement 💄 A new enhancement to functionality feature 🆕 A feature request dependencies Pull requests that update a dependency file labels Oct 13, 2024
Copy link

codecov bot commented Oct 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (9e2e830) to head (9b82983).
Report is 91 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##            master      #178     +/-   ##
===========================================
  Coverage   100.00%   100.00%             
===========================================
  Files          156       165      +9     
  Lines          426      5233   +4807     
===========================================
+ Hits           426      5233   +4807     
Flag Coverage Δ
unittests ?

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

Files with missing lines Coverage Δ
src/array/map.js 100.00% <100.00%> (ø)
src/object/amend.js 100.00% <100.00%> (ø)
src/object/objOf.js 100.00% <100.00%> (ø)
src/string/fuzzySearch.js 100.00% <100.00%> (ø)

... and 155 files with indirect coverage changes

@dhershman1 dhershman1 merged commit 18b0779 into master Oct 14, 2024
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file enhancement 💄 A new enhancement to functionality feature 🆕 A feature request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant