Skip to content

fix(scripts): keep headless Chrome alive on CI runners - #8

Merged
gitgusilva merged 1 commit into
gitgusilva:mainfrom
HermesSantos:feat/fix-preview-ci-chrome
Jul 30, 2026
Merged

fix(scripts): keep headless Chrome alive on CI runners#8
gitgusilva merged 1 commit into
gitgusilva:mainfrom
HermesSantos:feat/fix-preview-ci-chrome

Conversation

@HermesSantos

Copy link
Copy Markdown
Contributor

Add --no-sandbox and --disable-dev-shm-usage so Chromium does not SIGABRT under GitHub Actions, and surface stderr when it still fails.

Theme submission

Theme

  • Name:
  • Id (folder themes/<id>/):
  • Type: light / dark
  • Author (the name to credit, as you spell it):
  • Author URL (https link to your profile or the theme's repo):
  • Source / original author (for ports):

Description

Checklist

  • One theme added or changed in this PR
  • Files live in themes/<id>/ and id matches the folder name
  • id is a unique lowercase kebab-case slug
  • themes/<id>/theme.json validates against schema/theme.schema.json
  • All fifteen colors present as #RRGGBB
  • meta.version is semantic; meta.author is your name (not a login)
  • meta.authorUrl, if present, is an https link
  • themes/<id>/README.md added (with the preview@2x.png image)
  • themes/<id>/preview@2x.png added (node scripts/gen-previews.mjs <id>)
  • node scripts/validate.mjs reports no errors
  • Original author credited (for ports)

Add --no-sandbox and --disable-dev-shm-usage so Chromium does not
SIGABRT under GitHub Actions, and surface stderr when it still fails.

Co-authored-by: Cursor <cursoragent@cursor.com>

@gitgusilva gitgusilva left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Confirmed against the failing run on #6: headless Chromium aborts with SIGABRT on the runner, which these flags fix. Piping stderr through on failure is a nice touch — thanks!

@gitgusilva
gitgusilva merged commit 27b7a23 into gitgusilva:main Jul 30, 2026
2 checks passed
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.

2 participants