discordjs / discord.js-modules

Modularisation of discord.js (WIP)
Apache License 2.0
193 stars 30 forks source link

refactor: use the `node:` protocol #69

Closed almeidx closed 2 years ago

almeidx commented 2 years ago

Please describe the changes this PR makes and why it should be merged:

Status and versioning classification:

almeidx commented 2 years ago

Jest added support for the node protocol in 27.1.0

codecov[bot] commented 2 years ago

Codecov Report

Merging #69 (2877b15) into main (65cb8e2) will increase coverage by 3.64%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #69      +/-   ##
==========================================
+ Coverage   85.84%   89.48%   +3.64%     
==========================================
  Files           8        8              
  Lines        1010     1037      +27     
  Branches      101      107       +6     
==========================================
+ Hits          867      928      +61     
+ Misses         42        3      -39     
- Partials      101      106       +5     
Impacted Files Coverage Δ
packages/rest/src/lib/REST.ts 95.58% <100.00%> (+0.12%) :arrow_up:
packages/rest/src/lib/RequestManager.ts 93.68% <100.00%> (+11.02%) :arrow_up:
...ackages/rest/src/lib/handlers/SequentialHandler.ts 83.10% <100.00%> (ø)
packages/rest/src/lib/utils/constants.ts 100.00% <0.00%> (ø)
packages/rest/src/lib/CDN.ts 87.09% <0.00%> (+0.34%) :arrow_up:
packages/rest/src/lib/errors/DiscordAPIError.ts 79.76% <0.00%> (+3.57%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 65cb8e2...2877b15. Read the comment docs.