NuGet/Home

Multiple warnings on push are improperly formatted

Open

#8 001 ouverte le 14 avr. 2019

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)HTML (292 forks)batch import
Functionality:PushPriority:2Type:Buggood first issuehelp wanted

Métriques du dépôt

Stars
 (1 459 stars)
Métriques de merge PR
 (Merge moyen 464j 23h) (1 PR mergée en 30 j)

Description

I used dotnet.exe v2.2.202 to push a package that has no license with an API key that expires in a day. The output was:

info : Pushing dotnet-develop.0.0.0-prerelease.nupkg to 'https://www.nuget.org/api/v2/package'...
info :   PUT https://www.nuget.org/api/v2/package/
warn : Your API key expires in 1 days. Visit https://www.nuget.org/account/apikeys to regenerate your API key.,All published packages should have license information specified. Learn more: https://aka.ms/deprecateLicenseUrl.
info :   Created https://www.nuget.org/api/v2/package/ 652ms
info : Your package was pushed.

Notice that the warning on line 3 has two messages joined by a period and a comma, but no spaces. The client should properly format warning messages returned by the server.

Guide contributeur

Direction de recherche
Examinez le dépôt NuGet.Client, en particulier le code qui gère les avertissements de réponse du serveur lors de l'envoi de packages. Recherchez la logique de concaténation de chaînes qui joint les messages d'avertissement. Un fichier candidat se trouve dans l'espace de noms NuGet.CommandLine ou NuGet.Commands. Recherchez où les avertissements sont analysés à partir de la réponse HTTP et formatés pour la sortie console.
Stack technique
csharp
Domaine
cli
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.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80