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

Remove p-queue #2254

Merged
merged 1 commit into from
Apr 19, 2023
Merged

Remove p-queue #2254

merged 1 commit into from
Apr 19, 2023

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.

1 participant