Skip to content

fix: use camelCase keys in publish log statement#40

Merged
Kaiohz merged 1 commit into
mainfrom
fix/bricks-publish-log-camelcase
May 21, 2026
Merged

fix: use camelCase keys in publish log statement#40
Kaiohz merged 1 commit into
mainfrom
fix/bricks-publish-log-camelcase

Conversation

@Kaiohz
Copy link
Copy Markdown
Collaborator

@Kaiohz Kaiohz commented May 21, 2026

Fix log statement in publish_section_version that was using snake_case keys instead of camelCase after the payload was changed.

@Kaiohz Kaiohz merged commit c9e4df5 into main May 21, 2026
1 check passed
@Kaiohz Kaiohz deleted the fix/bricks-publish-log-camelcase branch May 21, 2026 16:19
@Kaiohz
Copy link
Copy Markdown
Collaborator Author

Kaiohz commented May 21, 2026

Review

Changements analysés

Le fix corrige les clés du log statement dans publish_section_version :

  • project_unique_idprojectUniqueId
  • section_keysectionKey
  • workflow_idworkflowId

Analyse

Pourquoi c'est important : Le payload est maintenant en camelCase (probablement suite à une migration ou standardisation API), mais le log statement utilisait encore les anciennes clés snake_case. Ça donnait des logs avec None pour ces champs — inutiles pour le debugging.

Ce qui est bien :

  • ✅ Fix minimal et ciblé
  • ✅ Message commit clair (conventional commit)
  • ✅ Cohérence avec le reste du codebase

Score de qualité : 8/10

Points positifs :

  • Fix précis, pas de scope creep
  • Convention de nommage respectée
  • CI passée avant merge

Suggestions d'amélioration (-2 points) :

  1. Tests unitaires : Idéalement, un test devrait vérifier que les logs contiennent les bonnes valeurs. Exemple avec caplog pytest :
def test_publish_logs_correct_keys(caplog):
    # Vérifie que les clés camelCase sont bien loggées
  1. Sign commit : Le commit n'est pas signé (verified: false). Configurer GPG signing pour la sécurité.

Approuvé — Fix propre et nécessaire.

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