microsoft / microsoft/OpenAPI.NET

avoid regressions for linting improvements

Ouverte
#1,400 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Langage dominant
C#
Étoiles
1.6k
Forks
286
Merge moyen
6 h 38 min
PR mergées (30 j)
35

Description

@simoncropp thank you for all the linting and other improvements pull requests (for other readers #1365 through #1398)

I'd like to make sure we don't introduce regressions on those improvements in the future. Hence this issue.

I think we should be able to get all those checked on every PR with:

  • editor config with a well configured csharp section
  • dotnet format --verify-no-changes
  • editing one of the existing GHA workflows to run this command.

Thoughts? Is this something you'd be willing to submit a PR for once the linting PRs are merged?

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par examiner les workflows GitHub Actions existants et la configuration de l’éditeur du dépôt, puis exécutez localement dotnet format --verify-no-changes. C’est terminé lorsque les paramètres C# pertinents d’editorconfig et l’exécution de la vérification du formatage réussissent sur chaque pull request sans régression.

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

Évaluation

Stack technique
csharp, github-actions
Domaine
build-system, ci-cd, developer-experience
Type d'issue
Fonctionnalité
Difficulté
3/5
Temps estimé
1-2 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.