Pour permettre aux équipes qui contribuent au service src non principal aux projets de la fédération l'ensemble du processus d'examen, les projets suivants ont été créés : avec des droits de +2 ouverts pour toutes les équipes. Cela soulage l'équipe de base de la fédération de la charge de toutes les revues de code non liées au framework de base, permettant aux autres équipes d'itérer plus rapidement dans leur cycle d'examen.
L'attribut src non principal est défini comme du code qui n'est pas requis par la fédération. qu'elle soit fonctionnelle (tests personnalisés, configurations, tests spécifiques, etc.). utilitaires).
IMPORTANT : Le tag src non principal ne doit pas étendre les classes de la fédération de base. Cela aura un impact sur les opérations de refactorisation/nettoyage futures. Si vous n'êtes pas sûr que votre code appartient à la suite principale ou à votre contribution, contactez android-tradefed@ pour en savoir plus. L'équipe centrale de la fédération est heureuse de conseiller et de recevoir vos demandes de fonctionnalités.
Par exemple, n'importe quelle classe de la catégorie Exemples de packages
com.google.android.tradefed.build:com.google.android.tradefed.build.LaunchControlProviderN'hésitez pas à nous contacter si vous ne savez pas ce que nous considérons comme un élément principal vers android-tradefed@.
[TOC]
Projets de contribution chemins
À qui s'adressent ces projets de contribution ?
Si vous travaillez actuellement dans les projets de la fédération, écrivez des tests/tests utilitaires/configurations, ces projets ont été créés pour vous.
Révisions de code dans les projets de contribution
L'objectif des projets de contribution est de vous permettre de développer dans Tradefed sans avoir besoin de l'examen de l'équipe principale (android-tradefed@). Nous nous attendons donc votre équipe ou toute personne connaissant votre contexte pour effectuer les revues de code vos CL.
N'hésitez pas à contacter android-tradefed@ en cas de blocage ou si vous avez besoin de conseils un cas particulier, mais ne vous y fiez pas par défaut pour réviser le code contrib. L'équipe Tradefed n'a aucun SLO concernant les revues de code dans les contributions.
Par où puis-je commencer à travailler sur ces projets ?
Elles ont été ajoutées aux fichiers manifestes de branche suivants et font déjà partie de la plate-forme principale. Si vous ne voyez pas les projets dans votre environnement, vous n'allez pas y contribuer.
- principal
- échangé
- oc-dev
- oc-dev-plus-aosp
- nyc-mr2-dev-plus-aosp
- développeur-daydream main
- principal-sans-fournisseur
- wear-main
Développement et tests
Le niveau de qualité du développement dans les contributions doit être le même que partout ailleurs dans les dépôts Android:
- Respecter les consignes de mise en forme Android
- Le code est testé et envoyé avec des tests
- La conception est réfléchie et a du sens
Comment puis-je effectuer un test en local dans une contribution ?
AOSP
Dans aosp, les tests unitaires des ressources platform/tools/tradefederation/contrib/tests/src/ et les tests unitaires doivent être ajoutés com.android.tradefed.prodtests.UnitTests dans les scripts de test pré-envoi et en local.
Scripts locaux Aosp après le déjeuner:
tools/tradefederation/core/tests/run_tradefed_aosp_presubmit.sh
Mise à jour des règles de compilation pour les projets de contribution
Les règles de compilation (makefiles) se trouvent dans leur propre dossier build/ et sont verrouillées par un
OWNERS fichier qui vous empêchera de les modifier sans examen supplémentaire
de l'équipe de base. Il s'agit de la seule limite applicable au projet de contribution.
Nous avons besoin de cet examen pour nous assurer qu'aucune dépendance inattendue n'est ajoutée au
l'ensemble des projets Tradefed sans le savoir ni l'accord de l'équipe de base.
Si vous avez vraiment besoin d'ajouter de nouvelles dépendances, veuillez contacter
android-tradefed@ pour examiner vos cas d'utilisation et vous conseiller.