Skip to content

fix(build): réparer la compilation après la montée du groupe gradle-all - #14

Merged
InstaZDLL merged 1 commit into
mainfrom
chore/bump-gradle-all
Aug 10, 2026
Merged

fix(build): réparer la compilation après la montée du groupe gradle-all#14
InstaZDLL merged 1 commit into
mainfrom
chore/bump-gradle-all

Conversation

@InstaZDLL

@InstaZDLL InstaZDLL commented Aug 9, 2026

Copy link
Copy Markdown
Owner

main ne compile plus. La montée du groupe gradle-all (5afdce6) a été fusionnée sans les ajustements que les nouvelles versions exigent : checkDebugAarMetadata échoue sur quatre dépendances.

Cette PR ne monte plus rien — main porte déjà toutes les versions. Elle ne contient que ce qu'il faut pour que ça reparte.

Ce qui bloque

4 issues were found when checking AAR metadata:
  androidx.core:core-ktx:1.19.0                        requires compileSdk ≥ 37
  androidx.core:core:1.19.0                            requires compileSdk ≥ 37
  androidx.lifecycle:lifecycle-viewmodel-compose:2.11.0 requires compileSdk ≥ 37
  androidx.lifecycle:lifecycle-runtime-compose:2.11.0   requires compileSdk ≥ 37

compileSdk 36.1 → 37. targetSdk reste à 36 : les deux réglages sont indépendants, et rien ici n'opte pour les nouveaux comportements d'exécution.

Deux points non bloquants, tant qu'on y est

createComposeRule passe en junit4.v2. L'ancien est déprécié par le nouveau BOM Compose — un avertissement à chaque compilation des tests. La v2 ordonnance en StandardTestDispatcher au lieu d'UnconfinedTestDispatcher ; la suite passe sans autre changement.

android.disallowKotlinSourceSets=false disparaît. KSP 2.3.11 ne déclare plus ses sources générées via kotlin.sourceSets, ce qui était toute la raison de ce réglage — son commentaire prévoyait déjà son retrait. Vérifié sur un clean : le build passe sans lui, et l'avertissement d'option expérimentale émis à chaque configuration part avec.

Et une nuisance de dépôt

gradlew.bat a été réenregistré avec des CRLF dans l'index, alors que .gitattributes (#10) demande LF stocké / CRLF déployé. Git le voit modifié en permanence et refuse tout changement de branche tant qu'on ne l'a pas remisé — c'est ce qui rendait la PR Dependabot précédente (#11) impossible à rebaser. Renormalisé ici. Le contenu déployé sur Windows est inchangé.

Validation

./gradlew clean testDebugUnitTest assembleDebug → BUILD SUCCESSFUL, 81 tests, 0 échec, aucun avertissement.

Toujours pas d'essai sur appareil réel. compileSdk 37 mérite un coup d'œil en conditions réelles avant publication.

https://claude.ai/code/session_01F89rkrDB9TxcwHbfgNoyY1

Summary by CodeRabbit

  • Documentation

    • Mise à jour des versions SDK minimale, cible et de compilation indiquées dans la documentation.
  • Améliorations

    • Compatibilité avec la nouvelle version du SDK Android, tout en conservant le niveau cible actuel.
    • Modernisation de l’exécution Gradle sous Windows.
    • Suppression d’une configuration de compatibilité devenue inutile.
  • Tests

    • Mise à jour des tests Compose vers la version 2 de l’API de test.

@github-actions github-actions Bot added scope: docs Docs, README, assets scope: project Build config, project files, tooling scope: tests Unit and UI tests type: chore Maintenance / tooling size: m 50-200 lines labels Aug 9, 2026
@coderabbitai

coderabbitai Bot commented Aug 9, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

Le projet distingue le SDK minimal, cible et de compilation. Le SDK de compilation passe à 37. Le test Compose utilise l’API v2. Le contournement Kotlin est supprimé. Le script Windows du wrapper simplifie l’exécution et la propagation des erreurs.

Changes

Mise à niveau Android et Gradle

Layer / File(s) Summary
Configuration Android et propriétés Gradle
app/build.gradle.kts, README.md, gradle.properties, app/src/test/...
compileSdk passe à 37, tandis que targetSdk reste à 36. La documentation reflète ces valeurs. Le contournement android.disallowKotlinSourceSets est supprimé. Le test utilise createComposeRule depuis Compose v2.
Script Windows du wrapper
gradlew.bat
L’initialisation du script et l’exécution de Gradle sont simplifiées. Les erreurs Java et le code de sortie final passent par %COMSPEC%.

Estimated code review effort: 2 (Simple) | ~10 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning La description est pertinente et détaillée, mais elle omet plusieurs sections requises du modèle, dont la checklist et les issues liées. Reprendre le modèle du dépôt et compléter les sections Summary, How I tested, Screenshots / clips, Checklist et Linked issues, avec « Closes #11 ».
✅ Passed checks (4 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed Le titre décrit clairement la correction principale de compilation liée à la mise à niveau du groupe gradle-all.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/bump-gradle-all

Comment @coderabbitai help to get the list of available commands.

Le bump fusionné en 5afdce6 ne compile pas : `checkDebugAarMetadata` échoue sur
quatre dépendances. Trois points à reprendre, dont un seul est bloquant.

`compileSdk` 36.1 → 37 : core-ktx 1.19 et lifecycle 2.11 refusent de se lier à
une API plus ancienne. `targetSdk` reste à 36 — les deux réglages sont
indépendants, et rien ici n'opte pour les nouveaux comportements d'exécution.

`createComposeRule` passe de `junit4` à `junit4.v2`, l'ancien étant déprécié
par le nouveau BOM Compose. La v2 ordonnance en `StandardTestDispatcher` au
lieu d'`UnconfinedTestDispatcher` ; la suite passe sans autre changement.

`android.disallowKotlinSourceSets=false` disparaît : KSP 2.3.11 ne déclare plus
ses sources générées via `kotlin.sourceSets`, ce qui était toute la raison de ce
réglage — son commentaire prévoyait déjà son retrait. Vérifié sur un `clean`, et
l'avertissement d'option expérimentale émis à chaque configuration part avec.

Renormalise enfin `gradlew.bat`, réenregistré avec des CRLF dans l'index par le
même commit alors que `.gitattributes` demande LF stocké / CRLF déployé. Git le
voyait modifié en permanence, ce qui bloque jusqu'au changement de branche. Le
contenu déployé sur Windows est inchangé.

Claude-Session: https://claude.ai/code/session_01F89rkrDB9TxcwHbfgNoyY1
@InstaZDLL
InstaZDLL force-pushed the chore/bump-gradle-all branch from cdcb490 to 0c031d7 Compare August 10, 2026 05:45
@InstaZDLL InstaZDLL changed the title chore(deps): monter la chaîne de build et les dépendances androidx fix(build): réparer la compilation après la montée du groupe gradle-all Aug 10, 2026
@InstaZDLL
InstaZDLL merged commit 4d75b19 into main Aug 10, 2026
2 checks passed
@InstaZDLL
InstaZDLL deleted the chore/bump-gradle-all branch August 10, 2026 05:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: docs Docs, README, assets scope: project Build config, project files, tooling scope: tests Unit and UI tests size: m 50-200 lines type: chore Maintenance / tooling

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant