discordjs / discord.js-modules

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

feat(SequentialHandler): add more info to debug when rate limit was hit #78

Closed ckohen closed 2 years ago

ckohen commented 3 years ago

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

Ported from https://github.com/discordjs/discord.js/pull/5801

Needs #77

Status and versioning classification:

codecov[bot] commented 3 years ago

Codecov Report

Merging #78 (750f18e) into main (c22081d) will decrease coverage by 0.09%. The diff coverage is 92.85%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #78      +/-   ##
==========================================
- Coverage   89.03%   88.94%   -0.10%     
==========================================
  Files           9        9              
  Lines        1487     1492       +5     
  Branches      153      155       +2     
==========================================
+ Hits         1324     1327       +3     
  Misses         12       12              
- Partials      151      153       +2     
Impacted Files Coverage Δ
...ackages/rest/src/lib/handlers/SequentialHandler.ts 82.19% <92.85%> (-0.25%) :arrow_down:

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 c22081d...750f18e. Read the comment docs.