Skip to content
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

[Feat]: Resolved issue #35 and improved bot.js #36

Closed
wants to merge 1 commit into from

Conversation

Yash-Vashishth
Copy link

Hi @jinx-vi-0 !! In accordance with issue #35 raised by me I have made the neccessary changes in the bot.js file .Improvements Made:

  1. Modularization: Split the messageCreate event handler into separate functions for each command.
  2. Error Handling: Improved error handling with more informative messages.
  3. Consistent Naming Conventions: Followed consistent naming conventions and code formatting.
  4. Code Formatting: Organized code for better readability.
  5. Caching: Cached LeetCode problems to reduce API calls.
  6. Documentation: Added inline comments for clarity.

Please merge my file into the main branch and also add the gssoc-ext , hacktoberfest and level3 labels to the pr.

Thanks,
Yash Vashishth
(Contributor, GSSoC'24 and hacktoberfest)

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

🎉 Thank you for your contribution! Your pull request has been submitted successfully. A maintainer from DDoL team will review it as soon as possible. We appreciate your support in making this project better

@sudo-dpkg sudo-dpkg closed this Oct 11, 2024
@Yash-Vashishth
Copy link
Author

Hi @jinx-vi-0!! Please atleast please merge this commit to the main branch and add labels. I have made many changes to the code

@sudo-dpkg sudo-dpkg added the duplicate This issue or pull request already exists label Oct 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants