Skip to content

Conversation

Vivan-1045
Copy link
Contributor

Related Issue

#154 - Node modules uploaded to the repository, security and performance risk

Description

This PR removes the node_modules directory from the repository to mitigate security risks, reduce repository size, and avoid version conflicts between dependencies. The directory is also added to .gitignore to prevent future accidental commits of the node_modules folder.

Type of PR

  • Bug fix
  • Feature enhancement
  • Documentation update
  • Other (specify): _______________

Screenshots / videos (if applicable)

Screenshot (11)

Checklist:

  • I have performed a self-review of my code
  • I have read and followed the Contribution Guidelines.
  • I have tested the changes thoroughly before submitting this pull request.
  • I have provided relevant issue numbers, screenshots, and videos after making the changes.
  • I have commented my code, particularly in hard-to-understand areas.

Additional context:

To restore dependencies, run npm install or yarn install to regenerate the node_modules folder.

Copy link

github-actions bot commented Apr 3, 2025

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our CONTRIBUTING.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

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.

Great job, @Vivan-1045! 🎉 Thank you for submitting your pull request. Your contribution is valuable and we appreciate your efforts to improve our project.

We will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check our contributing guidelines

@Vivan-1045 Vivan-1045 closed this Apr 3, 2025
@Vivan-1045 Vivan-1045 deleted the Vivan-1045/issue154 branch April 3, 2025 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant