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

VSCode instructions #198

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

VSCode instructions #198

wants to merge 4 commits into from

Conversation

amathew4538
Copy link

I added VSCode terminal instructions and the requirements file for it.

@amathew4538
Copy link
Author

P.S. This has only been tested on an M2 2022 Mac so far.

@PWhiddy
Copy link
Owner

PWhiddy commented Dec 13, 2024

Hey! thanks for sharing this!

For this to make sense to merge I think a couple things should be different.

  1. All pip requirements should be collected into a requirements.txt that can be installed in one go. IE, there shouldn't be any "pip install x" instructions besides the requirements.txt. If you have a working environment you can collect them all using pip freeze.

  2. There shouldn't instructions that are specific to a particular IDE. Just a requirements that works in one go for macos should be sufficient - basically just adding a macos_requirements.txt

@amathew4538
Copy link
Author

Sorry I didn't see this earlier. I was taking a break from coding. I have added MacOS requirements, and changed it to all IDEs. It does work in Pycharm and VS Code. I have also done the pip freeze.

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