Skip to content

add .qlty/qlty.toml suppressing similar-code on sibling product adapters#2

Merged
vzakharchenko merged 1 commit into
mainfrom
chore/qlty-suppress-adapter-similar-code
May 17, 2026
Merged

add .qlty/qlty.toml suppressing similar-code on sibling product adapters#2
vzakharchenko merged 1 commit into
mainfrom
chore/qlty-suppress-adapter-similar-code

Conversation

@vzakharchenko
Copy link
Copy Markdown
Member

The Jira / Confluence / Other product adapter pairs are intentionally symmetric thin classes: AbstractProductForgeAdapter and AbstractProductSelectAdapter hold the shared logic, and each concrete sibling only differs by the product-specific REST method or the forge-adapter instantiated in the constructor (2-4 lines). qlty:similar-code flags the identical import block and class shell as duplication (~40 of ~32 lines per file); further deduplication would require generics gymnastics or reflection and would hurt readability without removing real duplication.

The Jira / Confluence / Other product adapter pairs are intentionally symmetric
thin classes: AbstractProductForgeAdapter and AbstractProductSelectAdapter hold
the shared logic, and each concrete sibling only differs by the product-specific
REST method or the forge-adapter instantiated in the constructor (2-4 lines).
qlty:similar-code flags the identical import block and class shell as duplication
(~40 of ~32 lines per file); further deduplication would require generics
gymnastics or reflection and would hurt readability without removing real
duplication.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@sonarqubecloud
Copy link
Copy Markdown

@qltysh
Copy link
Copy Markdown

qltysh Bot commented May 17, 2026

Qlty


Coverage Impact

This PR will not change total coverage.

🚦 See full report on Qlty Cloud »

🛟 Help
  • Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

  • Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

  • File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

    • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

@vzakharchenko vzakharchenko merged commit 104a4a8 into main May 17, 2026
7 checks passed
@vzakharchenko vzakharchenko deleted the chore/qlty-suppress-adapter-similar-code branch May 17, 2026 15:13
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