discordjs / Commando

Official command framework for discord.js
Apache License 2.0
496 stars 242 forks source link

chore: mark optional peerDependencies as optional #278

Closed favna closed 3 years ago

favna commented 5 years ago

Please describe the changes this PR makes and why it should be merged:

A while ago a RFC was made for yarn to allow marking peer dependencies as optional and with that silencing the warnings about missing peer dependencies. You can read that RFC here. This was added in yarn in version 1.13.0 (PR and pnpm version 3.2.0-1 (PR). Later it was also added to npm in version 6.11.0 (Changelog)

Since it is now available in all 3 major package managing CLI tools I think this repo should make use of the additional fields in package.json to silence the warnings.

Note: a similar PR has been made to the main discord.js repository: https://github.com/discordjs/discord.js/pull/3511

Status

Semantic versioning classification: