-
Notifications
You must be signed in to change notification settings - Fork 6
Updated to Discord.js v14 #24
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Updated to Discord.js v14 #24
Conversation
"flexsearch": "^0.7.21", | ||
"glob": "^7.2.0", | ||
"node-fetch": "^2.6.6", | ||
"pm2": "^5.1.2" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why was pm2 removed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because it was installed but there is no reference to it in the code, nor is it referenced by any other dependency neither top level nor sublevel nor any script nor the dockerfile. In other words, I found no indication that it was being used. Saw no reason to include it when it isnt used.
Added a commit for discord.js-docs v 0.4.3 which was published earlier today, where an issue in that library that caused a top level error was resolved (see TeeSeal/discord.js-docs#30). I also changed the default query "Client" to "a", and moved this to constants.ts and named it |
…lt search query to mdn
Added DEFAULT_SEARCH_QUERY for mdn docs as well (this was already "a", but now refers to the same constant instead of just a inline char. Also moved the DJSAutocomplete from |
Uh oh!
There was an error while loading. Please reload this page.