Change `symbolize_names` to `symbolize_keys`

Ouverte
#341 9 commentaires 12 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
3/5
Temps estimé
1-2 jours
Accessibilité débutants
35/100
Type d'issue
Refactorisation
Clarté
Plutôt claire
Activité
À l'abandon
Stack technique
ruby
Domaine
tooling

Piste de recherche

Commencez par rechercher symbolize_names dans le dépôt, y compris dans sa documentation et ses tests, puis examinez comment l’option est analysée et exposée. Vérifiez si symbolize_keys est déjà utilisé ailleurs et définissez le périmètre de compatibilité ; le travail sera considéré comme terminé lorsque le nom convenu sera géré de manière cohérente et que les tests et la documentation concernés passeront.

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

Description

This occured to me while testing this option without the docs in front of me; it didn't work because I thought the option was named symbolize_keys (and only one day before I wrote the docs for it...). Why symbolize_names? symbolize_keys describes exactly what it does, symbolizing Hash keys, not names. Also, symbolize_keys already has some spread as a method name.

I will provide a PR if there is agreement on this.

Langage dominant
Ruby
Étoiles
596
Forks
223
Merge moyen
8 h 38 min
PR mergées (30 j)
2

Préparer son environnement

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.

Autres issues de ruby/psych

Toutes les issues de ruby/psych

Issues similaires

Plus d'issues Ruby

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.