Skip to content

Conversation

@NotSugden
Copy link
Contributor

Please describe the changes this PR makes and why it should be merged:
This PR updates the test files to use v12 methods instead of v11 methods

Status

  • Code changes have been tested against the Discord API, or there are no code changes
  • Haven't had time to test it all, but i believe it should work
  • I know how to update typings and have done so, or typings don't need updating

Semantic versioning classification:

  • This PR changes the library's interface (methods or parameters added)
    • This PR includes breaking changes (methods removed or renamed, parameters moved or removed)
  • This PR only includes non-code changes, like changes to documentation, README, etc.

Copy link
Member

@Jiralite Jiralite left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey! Also, it seems that in the file random.js, lines 98 to 102 lack the .cache to access the Collection!

@iCrawl iCrawl closed this Mar 1, 2020
@iCrawl iCrawl reopened this Mar 1, 2020
@iCrawl iCrawl merged commit 9b5f005 into discordjs:master Mar 7, 2020
@NotSugden NotSugden deleted the test branch September 14, 2020 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants