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

adding remote-dev containers type things #1

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

Conversation

fire-ant
Copy link

@fire-ant fire-ant commented Feb 9, 2021

I forked this to look at how api providers worked and thought it would be nice to add the dev container to hook up the tools listed in the README.

Ive added the necessary components to use the guide with a dev container as per https://code.visualstudio.com/docs/remote/containers. This saves you needing to prep an environment and is a decent cross-platform way of working with VSCode.

@TomyLobo
Copy link

If you still want them to merge this, you should probably redo your changes on a fresh branch instead of main.
A lot of changes unrelated to what you're trying to achieve here got mixed in.

@fire-ant
Copy link
Author

Thanks for the tip @TomyLobo - it was a fly by night offering and I guess I would've got some engagement if it created value and didn't incur more maintenance. No guarantees on the latter and I think there are more mature solutions out since then :)

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