Skip to content

Make lint-action go through the same script extensions for prettier#1351

Merged
matux merged 2 commits into
masterfrom
matux/ci-prettier
Oct 7, 2025
Merged

Make lint-action go through the same script extensions for prettier#1351
matux merged 2 commits into
masterfrom
matux/ci-prettier

Conversation

@matux

@matux matux commented Oct 7, 2025

Copy link
Copy Markdown
Contributor

Description of the change

This PR updates the prettier_extensions option in the CI workflow to follow the same extension list as our format script in package.json.

SDK-548/complete-prettier-rollout

@matux matux requested a review from Copilot October 7, 2025 16:23
@matux matux self-assigned this Oct 7, 2025

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

Updates CI workflow so lint-action's Prettier runs on the same set of file extensions as the local format script, ensuring consistency between local formatting and CI.

  • Adds prettier_extensions input to lint-action step
  • Aligns extension list with format script (js,cjs,mjs,md,json,yml,yaml)

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread .github/workflows/ci.yml
@matux matux force-pushed the matux/ci-prettier branch from f1b2e66 to 8d784b7 Compare October 7, 2025 16:38
@matux matux merged commit fba97dd into master Oct 7, 2025
5 of 6 checks passed
@matux matux deleted the matux/ci-prettier branch October 7, 2025 16:55
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