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

v0.18.0 #328

Merged
merged 12 commits into from
Sep 9, 2024
Merged

v0.18.0 #328

merged 12 commits into from
Sep 9, 2024

Conversation

daveshanley
Copy link
Member

Improved memory handling and performance in utility functions and component schema lookup.

This function was re-written in order to fix a number of performance issues with the original implementation.

Allocations were high and this function is used a lot, this new implementation is much
lighter on string allocations by using a string builder.
This function runs a lot and `syncMapToMap` is slow. The function is idepmotent so a cache prevents a lot of un-nessecary burn.
The function uses `equalFold` so this call is wasteful and slow.
This was stupid and I regret doing this. It causes tons of issues downstream.
Not sure how I missed these, however, better late than never.
stupid thing is throwing 404’s today for the latest release. Screw it, lock it down!
Copy link

codecov bot commented Sep 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.62%. Comparing base (7eb7079) to head (27351a8).
Report is 18 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #328      +/-   ##
==========================================
- Coverage   99.63%   99.62%   -0.02%     
==========================================
  Files         164      164              
  Lines       16597    16640      +43     
==========================================
+ Hits        16537    16578      +41     
- Misses         55       57       +2     
  Partials        5        5              
Flag Coverage Δ
unittests 99.62% <100.00%> (-0.02%) ⬇️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@daveshanley daveshanley merged commit ac1d206 into main Sep 9, 2024
3 of 4 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.

1 participant