Skip to content

ci: exclude VSCode and local GitHub CLI and Copilot config paths from scaffolded quality checks.#14

Merged
terabytesoftw merged 1 commit into
mainfrom
ci/exclude-vscode-github-cli
Jul 5, 2026
Merged

ci: exclude VSCode and local GitHub CLI and Copilot config paths from scaffolded quality checks.#14
terabytesoftw merged 1 commit into
mainfrom
ci/exclude-vscode-github-cli

Conversation

@terabytesoftw

Copy link
Copy Markdown
Contributor

Pull Request

  • Breaking change (fix or feature that would cause existing functionality to change)
  • Bugfix (non-breaking change that fixes an issue)
  • CI/build configuration
  • Documentation update
  • New feature (non-breaking change that adds functionality)
  • Refactoring (no functional changes)

@terabytesoftw terabytesoftw added the bug Something isn't working label Jul 5, 2026
@coderabbitai

coderabbitai Bot commented Jul 5, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 8adb1d38-16fc-4152-9140-31372577ee91

📥 Commits

Reviewing files that changed from the base of the PR and between 85c5d7f and 617d3fe.

📒 Files selected for processing (10)
  • .github/linters/.codespellrc
  • .github/linters/.editorconfig-checker.json
  • .gitignore
  • .prettierignore
  • CHANGELOG.md
  • metadata/.github/linters/.codespellrc
  • metadata/.github/linters/.editorconfig-checker.json
  • metadata/.gitignore
  • metadata/.prettierignore
  • metadata/.prettierrc.json
📜 Recent review details
🔇 Additional comments (12)
CHANGELOG.md (1)

10-11: LGTM!

.github/linters/.codespellrc (1)

2-2: LGTM!

metadata/.github/linters/.codespellrc (1)

2-2: LGTM!

.github/linters/.editorconfig-checker.json (1)

3-20: LGTM!

metadata/.github/linters/.editorconfig-checker.json (1)

3-20: LGTM!

.gitignore (3)

1-4: LGTM!


15-18: LGTM!


41-42: 🗄️ Data Integrity & Integration

.gitignore removal is intentional CHANGELOG.md already records removing phpunit.xml from .gitignore, and there’s no repo-local phpunit.xml to keep ignored.

			> Likely an incorrect or invalid review comment.
metadata/.gitignore (1)

1-4: LGTM!

Also applies to: 15-15, 36-38

.prettierignore (1)

1-27: LGTM!

metadata/.prettierignore (1)

4-10: LGTM!

metadata/.prettierrc.json (1)

9-9: LGTM!

Also applies to: 17-20


📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Updated ignore and linting rules for local AI, GitHub CLI, and Copilot-related configuration.
    • Expanded excluded paths for automated checks to reduce noise from generated, vendor, and local environment files.
    • Refined formatting settings to better target supported file types.
    • Revised changelog notes to reflect the updated quality-check exclusions.

Walkthrough

This PR updates linter and ignore configurations (Codespell, EditorConfig-checker, .gitignore, .prettierignore, .prettierrc.json) at the repo root and in metadata/, adding exclusions for local AI agent, editor, and GitHub CLI/Copilot config paths, simplifying Prettier overrides, and adding a corresponding CHANGELOG entry.

Changes

Scaffolded config exclusions

Layer / File(s) Summary
Codespell skip patterns
.github/linters/.codespellrc, metadata/.github/linters/.codespellrc
Expanded skip lists to include .github/* and .config/gh/* paths.
EditorConfig-checker exclude lists
.github/linters/.editorconfig-checker.json, metadata/.github/linters/.editorconfig-checker.json
Exclude arrays reordered and extended with new Copilot/agent/config exclusion patterns.
.gitignore updates
.gitignore, metadata/.gitignore
New local AI agent/editor/GitHub CLI config section added, Copilot comment renamed, and PHP tooling/PHPUnit ignore entries adjusted.
Prettier ignore and config updates
.prettierignore, metadata/.prettierignore, metadata/.prettierrc.json
Ignore lists trimmed/extended for GitHub-related paths; overrides narrowed to YAML-only glob and package.json override removed.
Changelog entry
CHANGELOG.md
Added bullet noting CI exclusions for VSCode and local GitHub CLI/Copilot config paths.

Estimated code review effort: 2 (Simple) | ~10 minutes

Sequence Diagram(s)

Not applicable — these are configuration/ignore-list changes without control-flow interactions.

Poem

A rabbit hops through configs neat,
Sweeping paths from Copilot's feet,
.gh and .claude tucked away,
No lint dust left to stray,
Hop, hop — the burrow's clean today! 🐇✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main CI config exclusion change in the pull request.
Description check ✅ Passed The description is related to the changeset and correctly marks the work as a refactoring.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ci/exclude-vscode-github-cli

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@terabytesoftw

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 617d3febd5

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread metadata/.prettierrc.json
Comment thread metadata/.gitignore
@terabytesoftw terabytesoftw merged commit c480fdd into main Jul 5, 2026
19 checks passed
@terabytesoftw terabytesoftw deleted the ci/exclude-vscode-github-cli branch July 5, 2026 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant