Skip to content

Comments

config: update github linguist exclusions#32

Merged
nberlette merged 2 commits intofeat/rewritefrom
main
Nov 26, 2025
Merged

config: update github linguist exclusions#32
nberlette merged 2 commits intofeat/rewritefrom
main

Conversation

@nberlette
Copy link
Owner

No description provided.

Copilot AI review requested due to automatic review settings November 26, 2025 10:43
Copy link

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 updates the .gitattributes file to expand GitHub Linguist exclusions, marking additional directories as vendored code to exclude them from language statistics. However, the changes include paths that don't exist in the current repository structure.

  • Adds three new linguist-vendored entries for src/lib/**, scripts/**, and crates/**
  • Two of the three new paths (src/lib/** and crates/**) reference non-existent directories

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

.vscode/** linguist-vendored
.github/** linguist-vendored
lib/** linguist-vendored
src/lib/** linguist-vendored
Copy link

Copilot AI Nov 26, 2025

Choose a reason for hiding this comment

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

The path src/lib/** appears to be incorrect. Based on the repository structure, only src/lib.rs exists (a file, not a directory). This pattern will not match anything and serves no purpose. Consider removing this line or verifying if a src/lib/ directory is expected in the future.

Suggested change
src/lib/** linguist-vendored

Copilot uses AI. Check for mistakes.
Repository owner deleted a comment from Copilot AI Nov 26, 2025
@nberlette nberlette merged commit 15b317d into feat/rewrite Nov 26, 2025
1 of 2 checks passed
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.

1 participant