search

RileCraft / DiscordBot-Template

A bot template with command , event , button , selectmenus , slashcommands handler and error handlers. Made in Discord.JS V14
MIT License
229 stars 59 forks source link

feat: @6.0.4 New Changes. #39

Closed Zaid-maker closed 2 years ago

Zaid-maker commented 2 years ago

i have updated package.json read the commits to know what i have done. Thank you

Zaid-maker commented 2 years ago

@RileCraft

RileCraft commented 2 years ago

First thanks for putting your time into this template, i want to keep required packages as low as possible and also i feel like it should be a choice on the user's side whether they want it or not as i personally also don't use nodemon and have seen many not do so as well. If they want nodemon or something like pm2 then they can just add it themself and use it according to their need.

Zaid-maker commented 2 years ago

Oh alright I will change it in the next commit.

On Mon, 14 Mar 2022, 8:25 pm RileCraft, @.***> wrote:

First thanks for putting your time into this template, i want to keep required packages as low as possible and also i feel like it should be a choice on the user's side whether they want it or not as i personally also don't use nodemon and have seen many not do so as well. If they want nodemon or something like pm2 then they can just add it themself and use it according to their need.

— Reply to this email directly, view it on GitHub https://github.com/RileCraft/DiscordBot-Template/pull/39#issuecomment-1066941478, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMXTCNDAEK4JOAN4K44NMG3U75K7LANCNFSM5QU5SQBQ . You are receiving this because you authored the thread.Message ID: @.***>

Zaid-maker commented 2 years ago

Well in my opinion most people use nodemon.

RileCraft commented 2 years ago

Depends on person

RileCraft commented 2 years ago

But anyway it might be considered in future but currently dont see it as required

Zaid-maker commented 2 years ago

Why close I was removing it ;-;

On Mon, 14 Mar 2022, 8:36 pm RileCraft, @.***> wrote:

But anyway it might be considered in future but currently dont see it as required

— Reply to this email directly, view it on GitHub https://github.com/RileCraft/DiscordBot-Template/pull/39#issuecomment-1066960728, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMXTCNEKN7N66HNH3H77G23U75MGJANCNFSM5QU5SQBQ . You are receiving this because you authored the thread.Message ID: @.***>

RileCraft commented 2 years ago

The other changes are very minor and can be delayed so eh

Zaid-maker commented 2 years ago

No minor changes those are important because that's dumb to install dependencies which already comes with nodejs

On Mon, 14 Mar 2022, 8:40 pm RileCraft, @.***> wrote:

The other changes are very minor and can be delayed so eh

— Reply to this email directly, view it on GitHub https://github.com/RileCraft/DiscordBot-Template/pull/39#issuecomment-1066969310, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMXTCNB5GNBJDWCMT3F7HQDU75MXPANCNFSM5QU5SQBQ . You are receiving this because you authored the thread.Message ID: @.***>