Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Upgrade discord.js to version 12 #555

Merged
merged 7 commits into from
Aug 9, 2020
Merged

Upgrade discord.js to version 12 #555

merged 7 commits into from
Aug 9, 2020

Conversation

Throne3d
Copy link
Collaborator

@Throne3d Throne3d commented Aug 9, 2020 •
edited
Loading

Also updates test stubs.

  • Manually test most main cases (pings of users + roles, emoji use, multi-channeling, join messages, webhooks)
  • Investigate hanging tests (looks to be due to Bot Events)

@Throne3d Throne3d added the dependencies Pull requests that update a dependency file label Aug 9, 2020
@Throne3d Throne3d self-assigned this Aug 9, 2020
@coveralls
Copy link

coveralls commented Aug 9, 2020 •
edited
Loading

Coverage Status

Coverage increased (+0.03%) to 96.884% when pulling d9fb80a on upgrade/discordjs-12 into ee2d70f on master.

@Throne3d Throne3d force-pushed the upgrade/discordjs-12 branch from 6a98615 to e1e01f1 Compare August 9, 2020 05:03
@Throne3d Throne3d force-pushed the upgrade/discordjs-12 branch from e1e01f1 to d9fb80a Compare August 9, 2020 05:05
@Throne3d Throne3d marked this pull request as ready for review August 9, 2020 05:05
@Throne3d Throne3d merged commit f047ce0 into master Aug 9, 2020
@Throne3d Throne3d deleted the upgrade/discordjs-12 branch August 9, 2020 05:07
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees

@Throne3d Throne3d

Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Throne3d @coveralls