Skip to content

Conversation

@raimund-schluessler
Copy link
Member

This PR removes the p-queue dependency. It does not work (well) with vue3 proxy usage (since it uses private members) and is also not really needed / brings more features then necessary. This also reduces the bundle size by 10 kByte.

@raimund-schluessler raimund-schluessler added 3 - to review dependencies Pull requests that update a dependency file labels Apr 19, 2023
@raimund-schluessler raimund-schluessler added this to the 0.15.0 milestone Apr 19, 2023
Signed-off-by: Raimund Schlüßler <raimund.schluessler@mailbox.org>
@codecov
Copy link

codecov bot commented Apr 19, 2023

Codecov Report

Merging #2254 (0b2568e) into master (a940bf2) will increase coverage by 0.05%.
The diff coverage is 31.57%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2254      +/-   ##
==========================================
+ Coverage   27.26%   27.32%   +0.05%     
==========================================
  Files          60       60              
  Lines        2919     2924       +5     
  Branches      743      744       +1     
==========================================
+ Hits          796      799       +3     
- Misses       2123     2125       +2     

@raimund-schluessler raimund-schluessler merged commit 979b9ab into master Apr 19, 2023
@delete-merged-branch delete-merged-branch bot deleted the fix/noid/remove-p-queue branch April 19, 2023 12:41
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants