Skip to content
This repository has been archived by the owner on Sep 22, 2022. It is now read-only.

Add a .devcontainer #78

Merged
merged 2 commits into from
May 17, 2022
Merged

Add a .devcontainer #78

merged 2 commits into from
May 17, 2022

Conversation

koddsson
Copy link
Contributor

@koddsson koddsson commented Feb 28, 2022

The kitchen sink image for Codespaces comes with node 14 and npm 6. We want to run node 16 and npm 8.

This PR creates a dockerfile and devcontainer to ensure that codespaces come with those newer versions.

Part of https://github.com/github/web-systems/issues/428

Co-authored-by: Ned Schwartz <[email protected]>
@koddsson koddsson requested a review from a team as a code owner February 28, 2022 17:10
@theinterned
Copy link
Contributor

CI is failing due to #77

I have another PR to skip that failing test #79 which it would be good to merge first.

@theinterned
Copy link
Contributor

hmmm I am noticing an issue when running tests in a Codespaces on this branch

03 03 2022 21:30:11.979:ERROR [launcher]: Cannot start ChromeHeadless
        /workspaces/details-dialog-element/node_modules/chromium/lib/chromium/chrome-linux/chrome: error while loading shared libraries: libnss3.so: cannot open shared object file: No such file or directory

It appears that the new container we're specifying is missing some dependencies we need to run chromium.
Screen Shot 2022-03-03 at 5 07 04 PM

Maybe we need to add more Codespaces features? But what?

Copy link
Contributor

@theinterned theinterned left a comment

Choose a reason for hiding this comment

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

This change is good! Though we should follow up with the same change in this PR github/auto-complete-element#88 to fix the issue with tests not running in codespaces.

@theinterned theinterned merged commit 5268f3e into main May 17, 2022
@theinterned theinterned deleted the node-16-in-codespaces branch May 17, 2022 20:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants