Skip to content

Add detekt-cli jar to .gitignore#3185

Merged
hamorillo merged 1 commit into
developfrom
hector.morilloprieto/addDetektCLItoGitIgnore
Feb 18, 2026
Merged

Add detekt-cli jar to .gitignore#3185
hamorillo merged 1 commit into
developfrom
hector.morilloprieto/addDetektCLItoGitIgnore

Conversation

@hamorillo

Copy link
Copy Markdown
Contributor

What does this PR do?

Adds a wildcard pattern (detekt-cli-*-all.jar) to .gitignore so that the detekt CLI jar required by the customDetektRules Gradle task is not committed to the repository.

Motivation

The customDetektRules task needs detekt-cli-*-all.jar in the root folder to run, but this is a large binary that should not be tracked in version control. Previously it was not ignored, risking accidental commits.

Additional Notes

The pattern uses a wildcard to cover all versions (e.g., detekt-cli-1.23.4-all.jar, detekt-cli-2.0.0-all.jar, etc.).

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Make sure you discussed the feature or bugfix with the maintaining team in an Issue
  • Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

@hamorillo
hamorillo changed the base branch from master to develop February 17, 2026 15:12
@datadog-datadog-prod-us1

This comment has been minimized.

@codecov-commenter

codecov-commenter commented Feb 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 71.27%. Comparing base (22c2444) to head (6c918df).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3185      +/-   ##
===========================================
- Coverage    71.28%   71.27%   -0.01%     
===========================================
  Files          929      929              
  Lines        34447    34442       -5     
  Branches      5816     5815       -1     
===========================================
- Hits         24553    24546       -7     
- Misses        8258     8263       +5     
+ Partials      1636     1633       -3     

see 47 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@hamorillo
hamorillo marked this pull request as ready for review February 17, 2026 20:05
@hamorillo
hamorillo requested a review from a team as a code owner February 17, 2026 20:05
The customDetektRules Gradle task requires detekt-cli-*-all.jar in the
root folder, but this binary should not be committed to the repository.
Adding a wildcard pattern ensures all current and future versions are ignored.
@hamorillo
hamorillo force-pushed the hector.morilloprieto/addDetektCLItoGitIgnore branch from ff38325 to 6c918df Compare February 18, 2026 06:56
@hamorillo
hamorillo merged commit b79d675 into develop Feb 18, 2026
26 checks passed
@hamorillo
hamorillo deleted the hector.morilloprieto/addDetektCLItoGitIgnore branch February 18, 2026 07:50
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