dwyl/english-words

Chore: Review all the open Pull Requests and Attempt to merge them 🧑‍💻

Open

#155 建立於 2022年11月10日

在 GitHub 查看
 (2 留言) (2 反應) (0 負責人)Python (9,687 star) (1,760 fork)batch import
T4hchorediscussenhancementhelp wantedpriority-3technical

描述

At present there are 20 open Pull Requests: https://github.com/dwyl/english-words/pulls

image

This is both a great and a really bad sign. 🎉 😕

On the positive side it's awesome that people want to contribute to the project [not just take the words and give nothing back...] and continually renews my faith in Open Source! 😍

But the down-side of having so many PRs open is that the efforts of several people are wasted ... 😞 ewe need to do a much better job of communicating how to contribute.

https://github.com/dwyl/english-words/pull/130 is an example of one that clearly took some effort and will add value to people! 👌 So we need to make every effort to ensure that the effort was not wasted! ⏳

Note: this is priority-3 ("nice to have") because while we definitely want to maintain and improve this repo, we have extremely limited time. And since we are not currently using it in our App ... we cannot justify the time allocation/investment right now. That's why I've "ignored" the open PRs. 🙄 Only features that relate to the App are given higher priority. It's possible that we could use English Words in the App e.g. for auto-completion/suggestion. In which case the priority of updating/maintaining this repo/project would be raised.

Todo

  • Review the PR: 👀

    • Assign it to yourself and add the label: in-review 👨‍🏫
    • View the code/words updates and leave comments 💬
    • If you feel it's immediately mergeable, approve it and assign it to me. 🙏
    • If it's not mergable, please leave a comment for the author (including an apology for the delay and link to this issue) 🔗
  • Criteria that qualifies for merging:

  • Instantly reject PRs that update Zip files and explain that Zip files files are commonly used for viruses so we cannot afford to trust anyone with updating them. "It's not you, it's me/us" we "Trust No One" https://en.wikipedia.org/wiki/Trust_no_one_(Internet_security)

@LuchoTurtle you mentioned this repo to me verbally in our catch up yesterday. If you want to make a stab at reviewing the open PRs in a Pomodoro Break, go for it! Please just leave a comment on this issue first and link to the PR you're picking off.

貢獻者指南