Faraday: Attendez que CodeClimate prenne en charge SimpleCov 0.18+, puis ajoutez à nouveau la collection de couverture de badge et de code

Créé le 1 avr. 2020  ·  5Commentaires  ·  Source: lostisland/faraday

Informations de base

  • Version Faraday : maître
  • Version Rubis : 2.6

Description du problème

Notre rapport de couverture de code à Codeclimate échoue toujours. Cela est dû au fait qu'ils ne prennent pas en charge le format 0.18+ de la sortie SimpleCov (couverture des branches, etc.).

J'ai reconnu la sortie d'erreur dans la 2ème étape du journaliste de codeclimate: un échec de Go pour analyser le nouveau format SimpleCov.

Error: json: cannot unmarshal object into Go struct field input.coverage of type []formatters.NullInt
Usage:
  cc-test-reporter after-build [flags]

~ Proposition : rétrograder simplecov vers une version supportée par codeclimate, et y garder un commentaire de code pour nous le rappeler.~

Proposition, mise à jour : Désactiver le reporting de couverture et le badge jusqu'à ce que Codeclimate supporte tout SimpleCov. Ce problème peut rester ouvert en tant que problème de suivi pour cela.

Étapes à reproduire

Voir cet échec des actions GH : https://github.com/lostisland/faraday/runs/543272466?check_suite_focus=true#step :9:8

Lorsque CI s'exécute au-delà de cette étape, ce programme est exécuté uniquement dans Ruby 2.6. Le code de sortie est 255 en cas d'échec, et la sortie ressemble à ce qui précède.

Tous les 5 commentaires

En y réfléchissant davantage, je veux en quelque sorte que le programme Go de CodeClimate ne retienne pas SimpleCov. Je choisirai plutôt de supprimer leur badge de couverture du README. Nous pouvons remettre le badge lorsqu'ils prennent en charge SimpleCov 0.18+.

Reporter problème ici :
https://github.com/codeclimate/test-reporter/issues/413

https://github.com/colszowka/simplecov/releases a beaucoup d'action en cours.

Une telle suppression entraînerait "la suppression des éléments CI qui transmettent également les données de couverture".

En effet, nous ne devrions pas rétrograder SimpleCov simplement parce qu'il n'est pas pris en charge par le test-reporter de CodeClimate. Accepter de désactiver le test-reporter jusqu'à ce qu'un correctif soit poussé

Nous avons supprimé codeclimate, fermant ceci

Cette page vous a été utile?
0 / 5 - 0 notes