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

Remove unnecessary files and dependencies #136

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Devasy23
Copy link

@Devasy23 Devasy23 commented Jun 3, 2024

Changes Made:

  1. Removed __pycache__ folders: Cleaned up unnecessary cache files from the repository to keep it tidy.
  2. Removed Python virtual environment: Deleted the virtual environment that was accidentally pushed into the repo by @revanth1718 in PR Streamlit App for Depression Support Issue no #99 #115 to avoid clutter and potential conflicts.
  3. Added .gitignore file: Included a .gitignore file to prevent accidental commits of unnecessary files such as cache files, virtual environments, and other temporary files.
  4. Applied black and isort formatting: Reformatted all Python and Jupyter Notebook files using black and isort to ensure code consistency and readability across the project.

These changes are part of my contribution to the GirlScript Summer of Code (GSSoC). Kindly assign this PR a level 2 and grant me the corresponding points.

Thank you!

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.

Welcome to Our repository.🎊 Thank you so much for taking the time to point this out.

@TAHIR0110 TAHIR0110 added level3 gssoc Associated with GSSOC labels Jun 6, 2024
@TAHIR0110 TAHIR0110 removed level3 gssoc Associated with GSSOC labels Jun 6, 2024
@Devasy23
Copy link
Author

Devasy23 commented Jun 7, 2024

I have completed my tasks for this pull request and it is ready for review and merging. Could you please assign the 'gssoc' tag and the appropriate level to it? I propose that it be classified as 'level 3'. Thank you.

@TAHIR0110

@TAHIR0110
Copy link
Owner

@Atharv714 kindly merge the PR.

@TAHIR0110
Copy link
Owner

@Devasy23 issue number?

@Devasy23
Copy link
Author

Devasy23 commented Aug 8, 2024

No linked issue, as it was almost stale for 2 months

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.

2 participants