Skip to content

Conversation

@didinele
Copy link
Member

Please describe the changes this PR makes and why it should be merged:
Should now work even with how we bundle files

@vercel
Copy link

vercel bot commented Oct 21, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Ignored Deployments
Name Status Preview Updated
discord-js ⬜️ Ignored (Inspect) Oct 27, 2022 at 7:38PM (UTC)
discord-js-guide ⬜️ Ignored (Inspect) Oct 27, 2022 at 7:38PM (UTC)

@codecov
Copy link

codecov bot commented Oct 21, 2022

Codecov Report

Merging #8776 (99715c7) into main (b2658ce) will not change coverage.
The diff coverage is 100.00%.

❗ Current head 99715c7 differs from pull request most recent head 9ec76f0. Consider uploading reports for the commit 9ec76f0 to get more accurate results

@@           Coverage Diff           @@
##             main    #8776   +/-   ##
=======================================
  Coverage   85.81%   85.81%           
=======================================
  Files          91       91           
  Lines        9254     9254           
  Branches     1125     1125           
=======================================
  Hits         7941     7941           
  Misses       1271     1271           
  Partials       42       42           
Flag Coverage Δ
brokers 65.24% <100.00%> (ø)
builders 100.00% <ø> (ø)
collection 100.00% <ø> (ø)
proxy 81.53% <ø> (ø)
rest 92.06% <ø> (ø)
util 100.00% <ø> (ø)
utilities 100.00% <ø> (ø)
voice 63.70% <ø> (ø)
ws 59.83% <ø> (ø)

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

Impacted Files Coverage Δ
packages/brokers/src/brokers/redis/BaseRedis.ts 51.16% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@kodiakhq kodiakhq bot merged commit e7cbc1b into discordjs:main Oct 27, 2022
@didinele didinele deleted the fix/brokers-script branch October 27, 2022 19:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants