Skip to content

Commit

Permalink
feat: Migrate to Discord.js v13 (#624)
Browse files Browse the repository at this point in the history
* chore: Use Node v16

* chore(deps): Update discord.js

* chore(deps): Install @discordjs/voice

* feat: Switch to Node.js v16

* feat(AboutCommand): Migrate to Discord.js v13

* feat(EvalCommand): Migrate to Discord.js v13

* feat(HelpCommand): Migrate to Discord.js v13

* feat(InviteCommand): Migrate to Discord.js v13

* feat(PingCommand): Migrate to Discord.js v13

* feat(main): Migrate to Discord.js v13

* feat(config): Remove compat layer

* feat(bot): Migrate to Discord.js v13

* feat(Util): Migrate to Discord.js v13

* feat(createEmbed): Migrate to Discord.js v13

* feat(CommandManager): Migrate to Discord.js v13

* feat(ReadyEvent): Migrate to Discord.js v13

* feat(PlayCommand): Migrate non music methods

* feat(MessageEvent): Migrate to Discord.js v13

* chore: Comment file that hasn't migrated yet.

* feat(ServerQueue): Migrate to Discord.js v13

* feat(Guild): Migrate to Discord.js v13

* fix(MessageEvent): MessageEvent -> MessageCreateEvent

* feat(PlayCommand): Migrate to Discord.js v13

* chore(PlayCommand): Add todo for Volume support

* feat(NowPlayingCommand): Migrate to v13

* feat(commands): Migrate to Discord.js v13

* fix(PlayCommand): Return if serverQueue is undefined

* feat(PlayCommand): Wait 15 seconds for the connection to be ready before playing

* refactor(Util): Refactor getGuildsCount, getChannelsCount. getUsersCount, and getTotalPlaying

* chore(Util): Fix typings

* fix(Util): Use fs.readFile instead of importing it in getPackageJSON

* fix(getPackageJSON): Fix getPackageJSON no valid main entry when trying to get jukebox's package.json

* fix(AboutCommand): Fix malformed AboutCommand message caused by indents

* chore: fix typings

* feat(MusicHelper): Migrate to Discord.js v13

* fix(MusicHelper): this -> that

* fix(downloader): Remove highWaterMark

We don't need it anymore, this is a workaround for #552
That error doesn't appear anymore on the new @discordjs/voice
And setting highWaterMark on the new voice system would cause lag
So I decided to delete the highWaterMark option instead.

* fix(ServerQueue): General fixes for music commands

* feat(VolumeCommand): Migrate to Discord.js v13

* fix(PlayCommand): Remove this listener as it conflicts with VoiceStateUpdateEvent

* fix(PlayCommand): Don't send Start playing message when unpaused

* feat(VoiceStateUpdateEvent): Migrate to Discord.js v13
  • Loading branch information
Hazmi35 authored Aug 16, 2021
1 parent acb0bf7 commit 6ccb421
Show file tree
Hide file tree
Showing 36 changed files with 3,857 additions and 532 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ jobs:
runs-on: ubuntu-latest
strategy:
matrix:
node-version: [">=12.9.0 <13.x", 14.x]
node-version: [">=16.6.0 <17.x"]
steps:
- name: Checkout repository
uses: actions/[email protected]
Expand All @@ -35,7 +35,7 @@ jobs:
runs-on: ubuntu-latest
strategy:
matrix:
node-version: [">=12.9.0 <13.x", 14.x]
node-version: [">=16.6.0 <17.x"]
steps:
- name: Checkout repository
uses: actions/[email protected]
Expand Down
1 change: 1 addition & 0 deletions .node-version
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
16
4 changes: 2 additions & 2 deletions Dockerfile
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
FROM node:14.17.5-alpine as build-stage
FROM node:16.6.2-alpine as build-stage

LABEL name "Jukebox (build stage)"
LABEL maintainer "Hazmi35 <[email protected]>"
Expand Down Expand Up @@ -28,7 +28,7 @@ RUN npm run build
RUN npm prune --production

# Get ready for production
FROM node:14.17.5-alpine
FROM node:16.6.2-alpine

LABEL name "Jukebox"
LABEL maintainer "Hazmi35 <[email protected]>"
Expand Down
Loading

0 comments on commit 6ccb421

Please sign in to comment.