-
Notifications
You must be signed in to change notification settings - Fork 42
changes for v1.1.0 #27
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
Conversation
|
✅ Deploy Preview for webcontainer-docs ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
@@ -131,7 +131,7 @@ async function bootWebContainer(terminal: import('xterm').Terminal) { | |||
|
|||
terminal.write('Booting WebContainer...'); | |||
|
|||
webcontainer = WebContainer.boot(); | |||
webcontainer = WebContainer.boot({ workdirName: 'demo' }); |
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 changes the folder name visible in the terminal from ~/webcontainersio-98a7w9
or whatever to simply ~/demo
.
(EDIT: nevermind, had to revert)
* The name of the working directory can be specified through `BootOptions#workdirName`. | ||
* The default value of the `PATH` environment variable is exposed in `WebContainer#path`. | ||
* The full path of the working directory is exposed in `WebContainer#workdir`. |
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.
Oh, lol, v1.1.0 broke some stuff, so we can't dogfood just yet. I'll revert for now. |
@@ -12,7 +12,7 @@ head: | |||
|
|||
This guide provides an overview on how to configure COOP/COEP headers, including hosting platform configuration. | |||
|
|||
WebContainers require that your page, even in development, be served with these two headers: | |||
WebContainers require that your page, even in development, is served with these two headers: |
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.
good catch, thanks!
Co-authored-by: Sylwia Vargas <[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.
thanks for the additions and editions 💙
Update docs for the new 1.1.0 version. What to do about API reference changes is TBD. I want to upgrade the docs themselves to use 1.1.0, but we might need to ship a fix before.