ADR-0019 cites a nonexistent test and omits the ones that pin its decisions
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 92/100
- Type d'issue
- Documentation
- Clarté
- Clairement spécifiée
- Activité
- Active
- Stack technique
- go
- Domaine
- documentation, testing-qa
Piste de recherche
Modifiez project-docs/adr/0019-azure-devops-as-a-second-transport.md directement. Exécutez go test ./internal/deps/ -list '.*' | grep -E 'ADO|ADR_0019|Survey' et comparez les tests listés avec l’ADR. C’est terminé lorsque le test inexistant est supprimé et que les cinq tests nommés dans l’issue sont inclus.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Follow-up from the review of #52.
The Consequences section of project-docs/adr/0019-azure-devops-as-a-second-transport.md lists tests that don't match the suite at 52a5544:
- It cites
TestRefresh_ADORepinDropsTheRecordedType, which does not exist. The re-pin behaviour is pinned byTestRefresh_ADORepinRetypesTheRefandTestRefresh_ADORepinToACommitTypesItAsACommit. - It leaves out tests that pin decisions the ADR records:
TestADR_0019_RefTypeIsRecordedOnlyWhereItIsNeeded(theref-typekey is recorded for ADO and omitted for GitHub)TestRefresh_ADOHeaderWithoutRefTypeResolvesIt(a header without the key is resolved on refresh)TestSurvey_ResolutionAbortsOnUnusableCLI(the ref-type lookup stops the batch when the CLI is unusable)
Check with:
go test ./internal/deps/ -list '.*' | grep -E 'ADO|ADR_0019|Survey'
Fix: make the ADR's test list match the suite. ADR-0019 isn't merged yet, so editing it in place is fine; the supersede-don't-edit rule applies only after it lands.
- Langage dominant
- Go
- Étoiles
- 34
- Forks
- 5
- Merge moyen
- 5 h 7 min
- PR mergées (30 j)
- 6
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de stacklok/modelith
-
enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 86/100
-
enhancement
Difficulté 2/5 Une demi-journée Accessibilité débutants 68/100
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 20/100
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 20/100
Toutes les issues de stacklok/modelith
Issues similaires
-
priority: low 🌱 type: enhancement 💅🏼
Difficulté 2/5 Une demi-journée Accessibilité débutants 84/100
nebari-dev/llm-serving-pack#199 ·
Les mainteneurs répondent en général sous 3 jours
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
-
area/helm kind/bug priority/backlog triage/accepted
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
lexfrei/cloudflare-tunnel-gateway-controller#889 ·
Les mainteneurs répondent en général sous 1 jour
-
bug difficulty: beginner documentation good first issue help wanted localization
Difficulté 1/5 Moins d'une heure Accessibilité débutants 90/100
wavefnd/wave-platform#140 ·
-
compiler/runtime
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
golang/go#81797 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour