Skip to content

Add .gitignore and .gitattributes for repo hygiene.#26

Merged
gustavofreze merged 1 commit intomainfrom
chore/repo-hygiene
Apr 21, 2026
Merged

Add .gitignore and .gitattributes for repo hygiene.#26
gustavofreze merged 1 commit intomainfrom
chore/repo-hygiene

Conversation

@gustavofreze
Copy link
Copy Markdown
Member

Prevent agent/IDE folders (.claude, .idea, .vscode) from being versioned and exclude dev-only files (tests, phpstan, phpcs, phpunit configs) from the Packagist tarball via export-ignore.

…ent/IDE folders (.claude, .idea, .vscode) from being versioned and exclude dev-only files (tests, phpstan, phpcs, phpunit configs) from the Packagist tarball via export-ignore.
Copilot AI review requested due to automatic review settings April 21, 2026 21:35
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 aims to improve repository hygiene by standardizing ignored local/dev artifacts and reducing what gets shipped in the Packagist dist archive via export-ignore.

Changes:

  • Reworked .gitignore to ignore common IDE/OS artifacts, Composer vendor output, and PHPUnit/coverage artifacts.
  • Simplified .gitattributes and added export-ignore rules intended to exclude dev-only files from Packagist archives.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 3 comments.

File Description
.gitignore Adds ignore rules for IDE folders, vendor output, and test/coverage artifacts.
.gitattributes Adds/updates export-ignore rules for Packagist dist and simplifies diff/text attributes.

Comment thread .gitattributes
Comment thread .gitattributes
Comment thread .gitattributes
@gustavofreze gustavofreze merged commit 8087d80 into main Apr 21, 2026
10 checks passed
@gustavofreze gustavofreze deleted the chore/repo-hygiene branch April 21, 2026 21:38
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