-
Notifications
You must be signed in to change notification settings - Fork 29
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
Config Gitpod 4-VS extensions #20
Conversation
.gitpod.yml
Outdated
code --install-extension ms-python.python | ||
code --install-extension aws-toolkit-vscode.aws-toolkit-vscode | ||
code --install-extension ms-mssql.mssql | ||
code --install-extension amazon-dynamodb.aws-dynamodb-toolkit |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thought: I don't think you need to do this for Gitpod, however I think it won't hurt. That said, vision wise I think this project should be agnostic of tools, like Gitpod, etc. So, to help us with that, could you move this to a shell script and invoke the script here instead? That will improve future portability.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great idea!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @Y4HYA4 !! :) - I've left a comment suggestion, but the PR adds value so I'll approve. If you want to have a go implementing my thoughts, go for it. Otherwise we can merge and address the changes in a separate issue / PR.
Closes: #17 |
Hey @loujaybee, any tweaks before the merge? |
The
tasks
includes thecode
command to install the specified VS Code extensions:The same list is included in the
extensions
para to ensure availability of the extensions when launching the workspace.