Skip to content

Add Dev Container configuration#462

Merged
shcheklein merged 2 commits intoiterative:masterfrom
ludelafo:master
May 15, 2023
Merged

Add Dev Container configuration#462
shcheklein merged 2 commits intoiterative:masterfrom
ludelafo:master

Conversation

@ludelafo
Copy link
Copy Markdown
Contributor

@ludelafo ludelafo commented Apr 3, 2023

This PR adds support for Dev Containers in order to have a quick and easy development environment using Dev Containers.

@ludelafo ludelafo requested a review from a team as a code owner April 3, 2023 11:11
Comment thread .devcontainer.json Outdated
@@ -0,0 +1,4 @@
{
"image": "mcr.microsoft.com/vscode/devcontainers/javascript-node:18",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's try mcr.microsoft.com/devcontainers/typescript-node:20

Copy link
Copy Markdown
Member

@shcheklein shcheklein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @ludelafo ! Let's also add:

    "codespaces": "yarn dev -H 0.0.0.0",
    "csps": "yarn dev -H 0.0.0.0",

to the package JSON (we do this in some other projects) to run the website in Codespace and enable access to it. Would be great to see if we need to update docs as well (contrib).

Comment thread .devcontainer.json Outdated
@@ -0,0 +1,4 @@
{
"image": "mcr.microsoft.com/vscode/devcontainers/javascript-node:18",
"postCreateCommand": "yarn"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add also:

  "customizations": {
    "vscode": {
      "extensions": [
        "redhat.vscode-yaml",
        "dbaeumer.vscode-eslint",
        "stkb.rewrap",
        "streetsidesoftware.code-spell-checker",
        "esbenp.prettier-vscode"
      ]
    }
  • they are handy when we deal with docs

@shcheklein shcheklein added enhancement New feature or request A: website Area: website housekeeping labels May 14, 2023
@ludelafo ludelafo requested a review from shcheklein May 15, 2023 15:20
@shcheklein shcheklein merged commit 92f2d34 into iterative:master May 15, 2023
@shcheklein
Copy link
Copy Markdown
Member

Thanks @ludelafo !

@github-actions
Copy link
Copy Markdown

Link Check Report

There were no links to check!

@ludelafo
Copy link
Copy Markdown
Contributor Author

You're welcome!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A: website Area: website enhancement New feature or request housekeeping

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants