googleapis / googleapis/google-cloud-cpp

Move microgenerator invocations to Bazel rules

Ouverte
#14,448 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
type: cleanup
Langage dominant
C++
Étoiles
659
Forks
462
Merge moyen
1 j 2 h
PR mergées (30 j)
89

Description

We use scripts to invoke the microgenerator via Bazel, for example:

https://github.com/googleapis/google-cloud-cpp/blob/34c4b4c3f61f716cb993c56dfd72486f427f1073/ci/cloudbuild/builds/generate-libraries.sh#L26-L33

It seems that kind of code really belongs in some kind of `genrule()` or a custom rule. There are many hard-coded magic strings in that call that can be converted to [`Label()`](https://bazel.build/rules/lib/builtins/Label#Label). That returns a `Label` object with all kinds of goodies: https://bazel.build/rules/lib/builtins/Label.html

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Commencez par ci/cloudbuild/builds/generate-libraries.sh autour des lignes 26-33 et examinez comment il invoque le microgenerator via Bazel. Déterminez si une genrule ou une règle personnalisée convient ; c’est terminé lorsque le script ne contient plus les détails d’invocation codés en dur et que Bazel prend en charge cette étape de génération.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Domaine
build-system
Type d'issue
Refactorisation
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.