Skip to content

feat: Add Detekt - #1348

Closed
Konstantin Pavlov (kpavlov) wants to merge 2 commits into
developfrom
kpavlov/detekt
Closed

Konstantin Pavlov (kpavlov) wants to merge 2 commits into
developfrom
kpavlov/detekt

Conversation

@kpavlov

@kpavlov Konstantin Pavlov (kpavlov) commented Jan 9, 2026

Copy link
Copy Markdown
Contributor

Add Gradle Detekt plugin and workflow

  • Added Detekt Gradle plugin
  • Added requirements and config for IDEA

Issues are to be fixed before enforcing the rules

Motivation and Context

To enforce code quality

Breaking Changes

No


Type of the changes

  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Tests improvement
  • Refactoring
  • CI/CD changes
  • Dependencies update

Checklist

  • The pull request has a description of the proposed change
  • I read the Contributing Guidelines before opening the pull request
  • The pull request uses develop as the base branch
  • Tests for the changes have been added
  • All new and existing tests passed

@kpavlov Konstantin Pavlov (kpavlov) changed the title core: Add Detekt chore: Add Detekt Jan 9, 2026
@kpavlov Konstantin Pavlov (kpavlov) changed the title chore: Add Detekt chores: Add Detekt Jan 9, 2026
@kpavlov Konstantin Pavlov (kpavlov) changed the title chores: Add Detekt chore: Add Detekt Jan 9, 2026
@kpavlov
Konstantin Pavlov (kpavlov) force-pushed the kpavlov/detekt branch 3 times, most recently from 4232bdf to 840796e Compare January 9, 2026 10:06
- Add gradle detekt plugin
- Update compilation workflow to include Deteck & optimize gradle
- Collect reports for compilation
Comment thread build.gradle.kts
config.from(rootProject.files("config/detekt.yml"))
basePath = rootProject.projectDir.absolutePath
parallel = true
ignoreFailures = true // TODO: fix issues and re-enable

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This is how to enable stricter checks: modelcontextprotocol/kotlin-sdk#507

@kpavlov

Copy link
Copy Markdown
Contributor Author

Briliantov Vadim (@Ololoshechkin), please review

@kpavlov Konstantin Pavlov (kpavlov) changed the title chore: Add Detekt feat: Add Detekt Feb 20, 2026
@kpavlov

Copy link
Copy Markdown
Contributor Author

At this time, there does not appear to be sufficient support for setting up static analysis.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request github-actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant