Skip to content

feat: use discord.js as dependency and use its client types#129

Merged
maybeanerd merged 4 commits intomainfrom
feat/add-discord.js-v14-support
Oct 2, 2022
Merged

feat: use discord.js as dependency and use its client types#129
maybeanerd merged 4 commits intomainfrom
feat/add-discord.js-v14-support

Conversation

@maybeanerd
Copy link
Member

@maybeanerd maybeanerd commented Oct 2, 2022 •
edited
Loading

BREAKING: drop support for discord.js v13 v12 and lower

@maybeanerd maybeanerd added dependencies Pull requests that update a dependency file hacktoberfest-accepted hacktoberfest-accepted labels Oct 2, 2022
@maybeanerd maybeanerd linked an issue Oct 2, 2022 that may be closed by this pull request
Add Support for discord.js v14 #90
Closed
@maybeanerd
Copy link
Member Author

maybeanerd commented Oct 2, 2022 •
edited
Loading

I still want to test if it would work to not bundle it as peer dependency, but as dev dependency - does it require discord.js to be built in the parent repo? It only uses types, so I would hope not. That could keep the package a lot smaller

@maybeanerd maybeanerd force-pushed the feat/add-discord.js-v14-support branch from c0a11a9 to 924f434 Compare October 2, 2022 11:03
@maybeanerd maybeanerd marked this pull request as ready for review October 2, 2022 11:10
@maybeanerd
Copy link
Member Author

maybeanerd commented Oct 2, 2022

tested it and we do not need to install the discord.js version in the parent project. even more, we can still use discord.js v13, which is nice. still a breaking change though

maybeanerd
maybeanerd commented Oct 2, 2022
View reviewed changes
@maybeanerd maybeanerd force-pushed the feat/add-discord.js-v14-support branch from 924f434 to 121997a Compare October 2, 2022 11:16
@maybeanerd
feat: install discord.js as dev dependency and directly use its clien…
b74e031 
…t types

BREAKING: drop support for discord.js v12 and lower

Tested typing using discord.js v13 and it was compatible
@maybeanerd maybeanerd force-pushed the feat/add-discord.js-v14-support branch from 121997a to b74e031 Compare October 2, 2022 11:16
@maybeanerd maybeanerd removed the hacktoberfest-accepted hacktoberfest-accepted label Oct 2, 2022
@maybeanerd
Copy link
Member Author

maybeanerd commented Oct 2, 2022 •
edited
Loading

removed hacktoberfest-accepted label until merged, because I think it's meant to be applied only after merging/approving the PR

@Andre601
Copy link

Andre601 commented Oct 2, 2022 •
edited
Loading

The PR only needs one of these:

  • Approving review
  • hacktoberfest-accepted label
  • Merge

So, if the PR is merged or you give it an approving review should it already count (Just needs to mature 7 days after this).

More info: https://hacktoberfest.com/participation/#pr-mr-details

@maybeanerd maybeanerd changed the title feat: install discord.js as peer dependency and directly use its client types feat: use discord.js as dependency and use its client types Oct 2, 2022
@maybeanerd
Copy link
Member Author

maybeanerd commented Oct 2, 2022

yeah that's the reason I removed the label, because hacktoberfest already thought this PR was approved and that's not true 😁

@maybeanerd maybeanerd merged commit 8574573 into main Oct 2, 2022
@maybeanerd maybeanerd deleted the feat/add-discord.js-v14-support branch October 2, 2022 18:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

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.

Add Support for discord.js v14

2 participants

@maybeanerd @Andre601