Plots2: Supprimer la variable inutilisée dans tag_controller_test.rb

Créé le 23 mars 2021  ·  3Commentaires  ·  Source: publiclab/plots2

Bonjour, il s'agit d'un problème réservé aux débutants . Cela signifie que nous avons travaillé pour le rendre plus lisible pour les personnes qui n'ont jamais contribué à notre base de code auparavant, ou même pour celles qui n'ont jamais contribué à l'open source auparavant .

Si tel est votre cas, nous souhaitons vous aider à faire le premier pas et pouvons répondre à vos questions et vous aider comme vous le faites. Notez que nous sommes particulièrement intéressés par les contributions de personnes issues de groupes sous-représentés dans les logiciels libres et open source !

Nous savons que le processus de création d'une pull request est le plus gros obstacle pour les nouveaux contributeurs. Ce numéro est pour vous

Si vous avez déjà contribué, pensez à laisser celui-ci à quelqu'un de nouveau et à consulter notre aide générale recherchés . Merci!

Ce que vous devez savoir.

Rien. Ce numéro est destiné à vous souhaiter la bienvenue dans l'Open Source :) Nous sommes heureux de vous guider tout au long du processus.

Pas à pas

  • [ ] 🙋 Réclamez ce problème : Commentez ci-dessous. Si quelqu'un d'autre l'a revendiqué, demandez-lui s'il a déjà ouvert une pull request et s'il est bloqué. Peut-être pouvez-vous l'aider à résoudre un problème ou à le déplacer !

Consultez cette page pour vous aider à faire vos premiers pas !

Ci-dessous, un "diff" montrant en rouge (et un - ) quelles lignes supprimer, et en vert (et un + ) quelles lignes ajouter :

https://github.com/publiclab/plots2/blob/d70efef8b13bdba3e39ab710f4a48b36c1778f12/test/functional/tag_controller_test.rb#L296

-tag = tags(:test)

https://github.com/publiclab/plots2/blob/d70efef8b13bdba3e39ab710f4a48b36c1778f12/test/functional/tag_controller_test.rb#L609

-user = UserSession.create(users(:admin))
+UserSession.create(users(:admin))

https://github.com/publiclab/plots2/blob/d70efef8b13bdba3e39ab710f4a48b36c1778f12/test/functional/tag_controller_test.rb#L617

-user = UserSession.create(users(:admin))
+UserSession.create(users(:admin))

https://github.com/publiclab/plots2/blob/d70efef8b13bdba3e39ab710f4a48b36c1778f12/test/functional/tag_controller_test.rb#L627

-user = UserSession.create(users(:admin))
+UserSession.create(users(:admin))
  • [] 💾 vos changements

  • [ ] 🔀 Lancer une Pull Request . Il existe deux manières de lancer une pull request :

  1. Si vous connaissez le terminal ou si vous souhaitez l'apprendre, voici un excellent tutoriel sur la façon d'envoyer une pull request à l'aide du terminal.

  2. Vous pouvez également modifier les fichiers directement dans votre navigateur et ouvrir une demande d'extraction à partir de là.

  • [ ] 🏁 Terminé Demandez un avis dans les commentaires :)

Veuillez nous tenir au courant

💬⏰ - Nous encourageons les contributeurs à être respectueux envers la communauté et à fournir une mise à jour

Si cela vous arrive, ne vous en faites pas ! Prenez un autre problème ouvert.

Est-ce que quelqu'un d'autre travaille déjà dessus ?

🔗- Nous encourageons les contributeurs à créer un lien vers le numéro d'origine dans leur pull request afin que tous les utilisateurs puissent facilement voir si quelqu'un a déjà commencé à le traiter.

👥- Si quelqu'un semble bloqué, proposez-lui de l'aide ! Sinon, jetez un œil à d'autres problèmes pour lesquels vous pouvez aider . Merci!

Des questions ?

Laissez un commentaire ci-dessous !

first-timers-only help wanted

Tous les 3 commentaires

Je vais prendre ce problème :raising_hand :

est ce que ce problème a été résolu?

@anirudhprabhakaran3
Non, mais PR est ouvert. (#9376)

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