Skip to content

Conversation

@ShiroTohu
Copy link
Contributor

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Breaking Change
  • Documentation Update

Description

I saw that .vscode was in the root structure of the project. I thought this was odd because to my knowledge, the .vscode folder is meant to store the settings for the User and Workplace environments. Also a bit worried that the folder would somehow end up on live. Therefore, I have done these two things.

  • Removed .vscode/ from repository
  • Added .vscode/ to .gitignore

Hope these small changes can help you out!
bouncing cat

Added/updated tests?

We encourage you to keep the code coverage percentage at 80% and above.

  • Yes
  • No, and this is why: minor change that doesn't need any tests.
  • I need help with writing tests

Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@caffeine-addictt caffeine-addictt added Type: Enhancement Suggest an improvement for an existing feature. Priority: Low - Task is considered lower-priority. labels Dec 11, 2023
@caffeine-addictt
Copy link
Member

Hi! Thanks for contributing @ShiroTohu.
Release builds only the src/thread/ directory so the .vscode/ directory will not be included in live.

However, yea, it is something i forgot to remove. Thanks for pointing it out 😄

@caffeine-addictt caffeine-addictt merged commit 98dc961 into python-thread:main Dec 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Priority: Low - Task is considered lower-priority. Type: Enhancement Suggest an improvement for an existing feature.

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants