Skip to content

Conversation

@solnic
Copy link
Collaborator

@solnic solnic commented Jan 9, 2026

#skip-changelog

@solnic solnic force-pushed the devcontainer-update branch from 8b8c9c8 to 0c9231c Compare January 9, 2026 15:45
@solnic solnic changed the title chore(devcontainer): use node from features chore(devcontainer): use node from features + update images Jan 9, 2026
@solnic solnic marked this pull request as ready for review January 9, 2026 15:45
&& apt-get clean \
&& rm -rf /var/lib/apt/lists/*

RUN curl -fsSL https://deb.nodesource.com/setup_22.x | bash -
Copy link

Choose a reason for hiding this comment

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

Chromium removed from Dockerfile breaks e2e tests

High Severity

The Dockerfile previously installed chromium and chromium-driver via apt-get, which are required by the Selenium-based e2e tests. These packages were removed but not replaced with a devcontainer feature. The test suite in spec/spec_helper.rb explicitly configures Capybara to use selenium_headless_chrome and references /usr/bin/chromium as the browser binary. The e2e tests will fail because Chromium is no longer available in the container.

Fix in Cursor Fix in Web

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

No it does not. We install them in the action.

@solnic solnic merged commit 43bb257 into master Jan 9, 2026
138 checks passed
@solnic solnic deleted the devcontainer-update branch January 9, 2026 16:07
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.

3 participants