-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Devcontainer support #3845
Devcontainer support #3845
Conversation
related to #3528 |
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.
This looks like a great contribution to me.
I made a few comments. Once those are addressed, this should be good to merge in.
Thanks for the PR @cychitivav!
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.
A few more comments.
Co-authored-by: Audrow Nash <[email protected]>
Co-authored-by: Tomoya Fujita <[email protected]>
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.
lgtm with the final confirmation.
i want to have the final confirmation on #3845 (comment), if that works for you, i think this is good to go. |
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.
LGTM!
Thanks @cychitivav for the PR and @fujitatomoya for the iteration! |
I have added the configurations to use GitHub Codespaces, which makes contributions easier as the only prerequisite is to have a GitHub account. Additionally, within this container, you can preview the HTML and everything in the cloud.