envoyproxy/envoy

`schema_validator_tool` should not be `testonly`

Open

#26 390 ouverte le 28 mars 2023

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)C++ (5 373 forks)batch import
area/releasearea/schema_validatorbughelp wanted

Métriques du dépôt

Stars
 (27 997 stars)
Métriques de merge PR
 (Merge moyen 8j) (378 PRs mergées en 30 j)

Description

Currently the schema_validator_tool is built using envoy_cc_test which marks it testonly and it in turn depends on a bunch of test libs that are likewise testonly

this should not be the case given that this tool is published, and means that it cannot be depended upon as a build target by other non-test build targets

i made an inexpert attempt at doing this, but found myself copying out increasingly large chunks of the test framework. I gave up when i needed to start copying out the test thread code

Guide contributeur

Direction de recherche
Modifiez les regles de construction pour schema validator tool afin de ne pas utiliser envoy cc test, mais d'utiliser une regle non testonly, tout en refactorisant les dependances pour eviter les dependances testonly.
Stack technique
cpp
Domaine
backendbuild system
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++Bazel
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40