Skip to content

Fixup Linguist stats as show on GitHub#40

Merged
agustingroh merged 1 commit intoscanoss:mainfrom
sschuberth:fixup-linguist-stats
Apr 29, 2026
Merged

Fixup Linguist stats as show on GitHub#40
agustingroh merged 1 commit intoscanoss:mainfrom
sschuberth:fixup-linguist-stats

Conversation

@sschuberth
Copy link
Copy Markdown
Contributor

@sschuberth sschuberth commented Apr 28, 2026

This avoids showing this Java project as a C project due to test data.

See 1 for reference.

Summary by CodeRabbit

  • Chores
    • Updated repository language detection configuration.

This avoids showing this Java project as a C project due to test data.

See [1] for reference.

[1]: https://github.com/github-linguist/linguist/blob/main/docs/overrides.md
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 28, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 0d18c51d-2701-4c1d-89a1-211a3062587f

📥 Commits

Reviewing files that changed from the base of the PR and between b996ae8 and 8afb0b4.

📒 Files selected for processing (1)
  • .gitattributes

📝 Walkthrough

Walkthrough

Adds a .gitattributes configuration rule that designates all files in the /testing/data/ directory as data-language files for GitHub Linguist classification purposes.

Changes

Cohort / File(s) Summary
Git Configuration
.gitattributes
Added linguist language attribute rule to classify files in /testing/data/** as data language.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A tiny rule in git's attire,
Makes data files stand in their proper fire,
GitHub now knows what dwells within,
The testing grounds where data spin,
Linguist smiles at clarity's win! 📊

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly addresses the main change: fixing Linguist statistics on GitHub by excluding test data from language classification.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Warning

Review ran into problems

🔥 Problems

Git: Failed to clone repository. Please run the @coderabbitai full review command to re-trigger a full review. If the issue persists, set path_filters to include or exclude specific files.


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 and usage tips.

@sschuberth
Copy link
Copy Markdown
Contributor Author

Hmm, not sure about the test failures @agustingroh. They seem to be unrelated to my PR, as test pass locally for me.

Copy link
Copy Markdown
Contributor

@agustingroh agustingroh left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution

@agustingroh agustingroh merged commit 13a33cb into scanoss:main Apr 29, 2026
1 of 2 checks passed
@sschuberth sschuberth deleted the fixup-linguist-stats branch April 29, 2026 10:32
@tatysanic-maker22
Copy link
Copy Markdown

Thank you for your contribution @sschuberth 😁

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