Skip to content

Add Biome linter and CI for actions/ - #168

Open
zeexzeex wants to merge 1 commit into
nodejs:mainfrom
zeexzeex:feat/add-biome-linter
Open

Add Biome linter and CI for actions/#168
zeexzeex wants to merge 1 commit into
nodejs:mainfrom
zeexzeex:feat/add-biome-linter

Conversation

@zeexzeex

Copy link
Copy Markdown

Summary

  • Adds Biome (@biomejs/biome@2.5.6) to actions/ with lint/format/ci scripts, following the discussion on Add linters #22 (Biome preferred, as in amaro / userland-migrations).
  • Adds .github/workflows/lint.yml to run yarn ci on PRs and pushes to main (same Corepack + Yarn pattern as existing workflows).
  • Applies Biome formatting/lint fixes to existing actions/ JS so CI passes; no intentional behavior changes.

Fixes #22

Test plan

  • cd actions && yarn install --frozen-lockfile && yarn ci
  • yarn lint
  • Confirm the new Lint / Biome workflow passes on this PR

Made with Cursor

Introduce Biome for the actions/ JS package and run biome ci on PRs.
Fixes nodejs#22

Signed-off-by: Avocado <ujubongbong@gmail.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@zeexzeex
zeexzeex requested a review from a team as a code owner August 11, 2026 02:06

@AugustinMauroy AugustinMauroy left a comment

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.

LGMT !

jobs:
biome:
name: Biome
runs-on: ubuntu-latest

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.

Suggested change
runs-on: ubuntu-latest
runs-on: ubuntu-slim

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.

Add linters

2 participants