Skip to content

fix: exclude ECC bundle dirs from Prettier to fix linter CI#33

Merged
sidharthmirch merged 2 commits into
ecc-tools/chorus-1775581082825from
copilot/fix-linter-checks
Apr 9, 2026
Merged

fix: exclude ECC bundle dirs from Prettier to fix linter CI#33
sidharthmirch merged 2 commits into
ecc-tools/chorus-1775581082825from
copilot/fix-linter-checks

Conversation

Copy link
Copy Markdown

Copilot AI commented Apr 9, 2026

The "Lint and Format Check" CI was failing because Prettier attempted to format AI tool config files introduced by the ECC bundle — one of which (.claude/homunculus/instincts/inherited/chorus-instincts.yaml) contains a YAML structure that Prettier parses as a syntax error.

Changes

  • .prettierignore — added .agents, .claude, and .codex to exclude ECC bundle directories from Prettier checks

Copilot AI changed the title [WIP] Fix the failing linter checks fix: exclude ECC bundle dirs from Prettier to fix linter CI Apr 9, 2026
Copilot AI requested a review from sidharthmirch April 9, 2026 08:04
@sidharthmirch sidharthmirch marked this pull request as ready for review April 9, 2026 08:33
Copilot AI review requested due to automatic review settings April 9, 2026 08:33
@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@sidharthmirch sidharthmirch merged commit 2c929f7 into ecc-tools/chorus-1775581082825 Apr 9, 2026
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes CI “Lint and Format Check” failures by preventing Prettier from attempting to format ECC bundle AI-tool configuration directories that include YAML content Prettier can’t parse.

Changes:

  • Updated .prettierignore to exclude ECC bundle directories (.agents, .claude, .codex) from Prettier runs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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