Plots2: Hapus Variabel yang Tidak Digunakan di tag_controller_test.rb

Dibuat pada 23 Mar 2021  ·  3Komentar  ·  Sumber: publiclab/plots2

Hai, ini hanya masalah pemula . Ini berarti kami telah bekerja untuk membuatnya lebih mudah dibaca oleh orang-orang yang belum pernah berkontribusi pada basis kode kami sebelumnya, atau bahkan orang-orang yang belum pernah berkontribusi pada open source sebelumnya .

Jika itu Anda, kami tertarik untuk membantu Anda mengambil langkah pertama dan dapat menjawab pertanyaan serta membantu Anda seperti yang Anda lakukan. Perhatikan bahwa kami sangat tertarik pada kontribusi dari orang-orang dari kelompok yang kurang terwakili dalam perangkat lunak bebas dan sumber terbuka!

Kami tahu bahwa proses pembuatan pull request adalah hambatan terbesar bagi kontributor baru. Masalah ini untukmu

Jika Anda telah berkontribusi sebelumnya, pertimbangkan untuk meninggalkan yang satu ini untuk seseorang yang baru , dan melihat melalui bantuan umum kami

Apa yang perlu Anda ketahui.

Tidak. Masalah ini dimaksudkan untuk menyambut Anda di Open Source :) Kami dengan senang hati memandu Anda melalui prosesnya.

Langkah demi Langkah

  • [ ] Klaim masalah ini : Komentar di bawah. Jika orang lain telah mengklaimnya, tanyakan apakah mereka telah membuka permintaan tarik dan jika mereka buntu -- mungkin Anda dapat membantu mereka memecahkan masalah atau meneruskannya!

Lihat halaman ini untuk mendapatkan bantuan dalam mengambil langkah pertama Anda!

Di bawah ini adalah "diff" yang ditampilkan dalam warna merah (dan - ) baris mana yang harus dihapus, dan dalam warna hijau (dan + ) baris mana yang akan ditambahkan:

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))
  • [] 💾 Komit perubahan Anda

  • [ ] Mulai Permintaan Tarik . Ada dua cara untuk memulai permintaan tarik:

  1. Jika Anda familiar dengan terminal atau ingin mempelajarinya, berikut adalah tutorial yang bagus tentang cara mengirim pull request menggunakan terminal.

  2. Anda juga dapat mengedit file langsung di browser Anda dan membuka permintaan tarik dari sana.

  • [ ] Selesai Tanya di komentar untuk review :)

Harap tetap perbarui kami

- Kami mendorong kontributor untuk menghormati komunitas dan memberikan pembaruan dalam waktu seminggu setelah mengklaim masalah khusus pemula. Kami senang untuk tetap menetapkannya kepada Anda selama yang Anda butuhkan jika Anda memperbarui kami dengan permintaan untuk lebih banyak waktu atau bantuan, tetapi jika kami tidak melihat aktivitas apa pun seminggu setelah Anda mengklaimnya, kami dapat menugaskannya kembali untuk diberikan kepada seseorang lain kesempatan. Terima kasih sebelumnya!

Jika ini terjadi pada Anda, jangan khawatir! Ambil masalah terbuka lainnya.

Apakah orang lain sudah mengerjakan ini?

- Kami mendorong kontributor untuk menautkan ke masalah asli dalam permintaan tarik mereka sehingga semua pengguna dapat dengan mudah melihat apakah seseorang sudah memulainya.

- Jika seseorang tampak buntu, tawarkan bantuan! Jika tidak, lihat beberapa masalah lain yang dapat Anda bantu . Terima kasih!

Pertanyaan?

Tinggalkan komentar di bawah!

first-timers-only help wanted

Semua 3 komentar

Saya akan mengambil masalah ini

Apakah masalah ini telah diselesaikan?

@anirudhprabhakaran3
Tidak. Tapi, PR terbuka.(#9376)

Apakah halaman ini membantu?
0 / 5 - 0 peringkat