Skip to content

Conversation

@JayKamsu
Copy link

Objet

Corriger les violations de la règle Sonar RSPEC‑1940 (“Boolean checks should not be inverted”) afin de rendre la lecture du code plus naturelle et consensuelle.

Modifications clés

  1. SecStrucTools, BioAssemblyTools, Element, Author, Location, Equals, InsdcParser : remplacement des négations imbriquées (!(...)) par des tests affirmatifs explicites, ce qui clarifie l’intention de chaque condition.
  2. CeCalculatorEnhanced et CECalculator : réécriture des comparaisons sur nAtom avec une inégalité directe (>=) pour éliminer les doubles négations et faciliter la compréhension du seuil appliqué.
  3. Aucune logique métier modifiée ; il s’agit exclusivement d’améliorations de style pour se conformer à RSPEC‑1940 et harmoniser la lisibilité du code.

Copy link
Contributor

@josemduarte josemduarte left a comment

Choose a reason for hiding this comment

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

LGTM thanks!

@josemduarte josemduarte merged commit e6ba379 into biojava:master Dec 4, 2025
5 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.

2 participants