Update to 2.0.0
- .gitignore 0 additions, 3 deletions.gitignore
- ble/build.gradle 10 additions, 9 deletionsble/build.gradle
- ble/src/main/AndroidManifest.xml 2 additions, 2 deletionsble/src/main/AndroidManifest.xml
- ble/src/main/java/com/orange/proximitynotification/ProximityNotificationEvent.kt 32 additions, 0 deletions...range/proximitynotification/ProximityNotificationEvent.kt
- ble/src/main/java/com/orange/proximitynotification/ProximityNotificationEventId.kt 80 additions, 0 deletions...nge/proximitynotification/ProximityNotificationEventId.kt
- ble/src/main/java/com/orange/proximitynotification/ProximityNotificationLogger.kt 44 additions, 0 deletions...ange/proximitynotification/ProximityNotificationLogger.kt
- ble/src/main/java/com/orange/proximitynotification/ProximityNotificationService.kt 54 additions, 11 deletions...nge/proximitynotification/ProximityNotificationService.kt
- ble/src/main/java/com/orange/proximitynotification/ble/BleProximityNotificationFactory.kt 9 additions, 1 deletion...ximitynotification/ble/BleProximityNotificationFactory.kt
- ble/src/main/java/com/orange/proximitynotification/ble/advertiser/BleAdvertiserImpl.kt 41 additions, 9 deletions...proximitynotification/ble/advertiser/BleAdvertiserImpl.kt
- ble/src/main/java/com/orange/proximitynotification/ble/gatt/BleGattManagerImpl.kt 104 additions, 18 deletions...ange/proximitynotification/ble/gatt/BleGattManagerImpl.kt
- ble/src/main/java/com/orange/proximitynotification/ble/scanner/BleScannerImpl.kt 64 additions, 12 deletions...range/proximitynotification/ble/scanner/BleScannerImpl.kt
- ble/src/test/java/com/orange/proximitynotification/ble/scanner/ScanResultExtKtTest.kt 5 additions, 3 deletions.../proximitynotification/ble/scanner/ScanResultExtKtTest.kt
- build.gradle 4 additions, 4 deletionsbuild.gradle
- coreui/build.gradle 22 additions, 18 deletionscoreui/build.gradle
- coreui/src/main/java/com/lunabeestudio/stopcovid/coreui/UiConstants.kt 2 additions, 1 deletion...in/java/com/lunabeestudio/stopcovid/coreui/UiConstants.kt
- coreui/src/main/java/com/lunabeestudio/stopcovid/coreui/custom/BlockingContentLoadingProgressBar.kt 4 additions, 2 deletions...pcovid/coreui/custom/BlockingContentLoadingProgressBar.kt
- coreui/src/main/java/com/lunabeestudio/stopcovid/coreui/extension/StringExt.kt 4 additions, 36 deletions...com/lunabeestudio/stopcovid/coreui/extension/StringExt.kt
- coreui/src/main/java/com/lunabeestudio/stopcovid/coreui/extension/ViewExt.kt 4 additions, 2 deletions...a/com/lunabeestudio/stopcovid/coreui/extension/ViewExt.kt
- coreui/src/main/java/com/lunabeestudio/stopcovid/coreui/fragment/BaseFragment.kt 37 additions, 7 deletions...m/lunabeestudio/stopcovid/coreui/fragment/BaseFragment.kt
- coreui/src/main/java/com/lunabeestudio/stopcovid/coreui/fragment/FastAdapterFragment.kt 32 additions, 1 deletion...eestudio/stopcovid/coreui/fragment/FastAdapterFragment.kt
-
-
Bonjour,
ce commit (comme les précédents) ne correspond pas vraiment aux pratiques et standards en vigueur dans les projets de l'écosystème open source, et peut donner l'impression que l'équipe de développement n'utilise ou ne maîtrise pas l'outil git (confusion entre un commit et ce qui devrait être un tag).
En particulier, ce commit semble agréger de nombreux changements non reliés entre eux et non clairement documentés, avec un message de commit peu descriptif. Ces pratiques rendent difficile l'évaluation et l'audit du code, ainsi que la détection simple de problèmes (puisqu'il est impossible de bisecter leur apparition dans un mégacommit)
De nombreuses ressources sont disponibles sur le net pour se former à l'utilisation d'outils collaboratifs dans une optique ouverte, par exemple Git Best Practices ou How to write a git commit message.
Serait-il possible de corriger ces points à l'avenir ?
Merci d'avance !
-
Votre message est pertinent et courtois. L’absence de réaction m’interpelle. On peut comprendre que la team soit complètement mobilisée sur les ajustements d’implémentation mais on peut aussi s’interroger sur une utilisation de l’outil collaboratif GIT qui mériterait d’être revue pour être en conformité avec l’intention initiale. Restons attentifs à la réponse qui sera adressée à Marc Jeanmougin
-
-
mentioned in issue #52 (closed)