doc: Add template for good first issues #17339

pull michaelfolkson wants to merge 1 commits into bitcoin:master from michaelfolkson:20191101-first-issue changing 1 files +20 −0
  1. michaelfolkson commented at 12:59 PM on November 1, 2019: contributor

    closes #17317

    Attempted to address everyone's suggestions in #17317 without making it too long. The first half is for the benefit of the individual opening the issue and the second half is for the benefit of the new contributor. Ideally we don't want the second half to be deleted by the individual opening the issue but whether they delete the first half or not isn't really a concern

  2. fanquake added the label Docs on Nov 1, 2019
  3. in .github/ISSUE_TEMPLATE/good_first_issue.md:1 in d9ceae5aaa outdated
       0 | @@ -0,0 +1,11 @@
       1 | +The purpose of the good first issue label is to highlight which issues are suitable for a new contributor without a deep understanding of the codebase.
    


    MarcoFalke commented at 1:03 PM on November 1, 2019:

    Missing the header:

    ---
    name: Good fist issue
    about: (Regular devs only): Suggest a new good first issue
    title: ''
    labels: goog first issue
    assignees: ''
    
    ---
    

    michaelfolkson commented at 1:40 PM on November 1, 2019:

    Thanks! I will add this. GitHub seems to be in process of upgrading templates so I was uncertain whether it needed such a header. "You are using an old version of issue templates. Please update to the new issue template workflow"


    jonatack commented at 2:10 PM on November 1, 2019:

    nit: possibly enclose the label name in backticks (in this case, capitalise it like the tag).


    michaelfolkson commented at 2:40 PM on November 1, 2019:

    The tag isn't capitalized? good first issue works though


    MarcoFalke commented at 3:05 PM on November 1, 2019:

    The header is the new workflow.

    You can test this by making michaelfolkson:20191101-first-issue the default branch and enabling issues for your repo in the settings. Then, add the "good first issue" label from Bitcoin Core (needs to be capitalized in the same way)


    michaelfolkson commented at 4:38 PM on November 3, 2019:

    I've followed the above instructions but I can't seem to open a good first issue issue with the template that I've added as part of this PR.

    good first issue isn’t one of the options I see when creating a new issue (bug report, feature request, report a security vulnerability are the three buttons). “Edit templates” just directs me back to the ISSUE_TEMPLATE directory. Opening a blank issue and changing the label doesn’t update the template.

    I’m assuming the issue creation experience needs to be customized with a config.yml file? https://help.github.com/en/github/building-a-strong-community/configuring-issue-templates-for-your-repository


    MarcoFalke commented at 5:47 PM on November 3, 2019:

    You'll have to change the default branch in https://github.com/michaelfolkson/bitcoin/settings/branches


    michaelfolkson commented at 6:05 PM on November 3, 2019:

    I did. I followed the above instructions.

  4. in .github/ISSUE_TEMPLATE/good_first_issue.md:11 in d9ceae5aaa outdated
       6 | +
       7 | +Want to work on this issue?
       8 | +
       9 | +You do not need to request permission to start working on this. It is encouraged (but not compulsory) to comment on the issue if you are planning to work on it. This will help the maintainers of the project monitor which issues are actively being addressed and is also an effective way to request assistance if and when you need it.
      10 | +
      11 | +For guidance on opening a pull request please see [CONTRIBUTING.md](https://github.com/bitcoin/bitcoin/blob/master/CONTRIBUTING.md) 
    


    jonatack commented at 2:07 PM on November 1, 2019:

    Linter failed due to whitespace at the end of this line.


    jonatack commented at 2:08 PM on November 1, 2019:

    nit: possibly add a comma after "request" and a full stop at the end of the sentence.


    michaelfolkson commented at 2:41 PM on November 1, 2019:

    Ah my bad. Thanks


    fjahr commented at 3:01 PM on November 1, 2019:

    I think this can be stronger. I would almost like a disclaimer, like "[ ] Yes, I confirm I have read the contributer guidelines" ;)

    For guidance on opening a pull request and other helpful tips, please read [CONTRIBUTING.md](https://github.com/bitcoin/bitcoin/blob/master/CONTRIBUTING.md) before starting the process.
    
  5. in .github/ISSUE_TEMPLATE/good_first_issue.md:5 in d9ceae5aaa outdated
       0 | @@ -0,0 +1,11 @@
       1 | +The purpose of the good first issue label is to highlight which issues are suitable for a new contributor without a deep understanding of the codebase.
       2 | +
       3 | +Useful skills:
       4 | +
       5 | +(For example “C++11 std::thread”, “Qt5 GUI and async GUI design” or “basic understanding of Bitcoin mining and the Bitcoin Core RPC interface”)
    


    jonatack commented at 2:09 PM on November 1, 2019:

    nit: missing comma or colon after "example" and full stop at end of sentence.

  6. jonatack commented at 2:10 PM on November 1, 2019: member

    Concept ACK. A couple of minor nits; feel free to ignore modulo the linter.

  7. in .github/ISSUE_TEMPLATE/good_first_issue.md:9 in d9ceae5aaa outdated
       0 | @@ -0,0 +1,11 @@
       1 | +The purpose of the good first issue label is to highlight which issues are suitable for a new contributor without a deep understanding of the codebase.
       2 | +
       3 | +Useful skills:
       4 | +
       5 | +(For example “C++11 std::thread”, “Qt5 GUI and async GUI design” or “basic understanding of Bitcoin mining and the Bitcoin Core RPC interface”)
       6 | +
       7 | +Want to work on this issue?
       8 | +
       9 | +You do not need to request permission to start working on this. It is encouraged (but not compulsory) to comment on the issue if you are planning to work on it. This will help the maintainers of the project monitor which issues are actively being addressed and is also an effective way to request assistance if and when you need it.
    


    jnewbery commented at 2:57 PM on November 1, 2019:

    It is encouraged (but not compulsory) to comment on the issue if you are planning to work on it. This will help the maintainers of the project monitor which issues are actively being addressed and is also an effective way to request assistance if and when you need it.

    I don't think it's true that maintainers are actively monitoring which new contributors are working on which up for grabs issues. I'd just replace this with "You may wish to comment on this issue if you start working on it. You can also report progress or ask for help here."


    fjahr commented at 2:57 PM on November 1, 2019:

    Just a small suggestion for the second sentence.

    You do not need to request permission to start working on this. It is encouraged (but not compulsory) to comment on the issue if you are planning to work on it. This will help the maintainers of the project to monitor which issues are actively being addressed. Feel free to request assistance when you need it, rather than letting the issue go stale. When you do ask for help, consider linking to your branch or opening a draft PR.
    

    fjahr commented at 3:06 PM on November 1, 2019:

    I think it helps to encourage people to report in progress/issues more frequently so that we have less "XY, are you still working on this?".


    michaelfolkson commented at 3:27 PM on November 1, 2019:

    Yeah that was my thinking @fjahr. Even if it isn't strictly true (as @jnewbery says) that maintainers actively monitor this I still think it encourages the sought after behavior, namely let everyone (maintainers, author of issue, other new contributors) know you are planning to work on this and keep everyone updated with your progress. Not compulsory but actively encouraged.


    michaelfolkson commented at 4:36 PM on November 1, 2019:

    I'll keep this as it is unless @jnewbery comes back with strong reservations.


    jnewbery commented at 4:44 PM on November 1, 2019:

    My reservation is you're telling people that maintainers are monitoring this, which isn't true!


    michaelfolkson commented at 4:49 PM on November 1, 2019:

    Haha ok fair enough. How about replacing "the maintainers" with "other contributors"?

    "This will help other contributors monitor which issues are actively being addressed and is also an effective way to request assistance if and when you need it."


    fjahr commented at 4:54 PM on November 1, 2019:

    was just going to suggest something similar: s/This will help the maintainers of the project monitor.../This will help others interested in contributing to see.../

  8. fjahr commented at 3:01 PM on November 1, 2019: member

    Concept ACK

  9. michaelfolkson force-pushed on Nov 3, 2019
  10. in .github/ISSUE_TEMPLATE/good_first_issue.md:3 in 690684dd1b outdated
       0 | @@ -0,0 +1,20 @@
       1 | +---
       2 | +name: Good first issue
       3 | +about: (Regular devs only): Suggest a new good first issue
    


    MarcoFalke commented at 6:54 PM on November 3, 2019:
    about: '(Regular devs only): Suggest a new good first issue'
    

    This might need quotes for GitHub to pick it up


    michaelfolkson commented at 3:17 PM on November 5, 2019:

    Ok this works now for me. Thanks for the help @MarcoFalke.

  11. in .github/ISSUE_TEMPLATE/good_first_issue.md:18 in 690684dd1b outdated
      13 | +
      14 | +(For example, “C++11 std::thread”, “Qt5 GUI and async GUI design” or “basic understanding of Bitcoin mining and the Bitcoin Core RPC interface”.)
      15 | +
      16 | +Want to work on this issue?
      17 | +
      18 | +You do not need to request permission to start working on this. It is encouraged (but not compulsory) to comment on the issue if you are planning to work on it. This will help other contributors monitor which issues are actively being addressed and is also an effective way to request assistance if and when you need it.
    


    jonatack commented at 11:27 AM on November 4, 2019:

    nit: could use active friendly voice rather than passive formal, e.g. "We encourage you" rather than "It is encouraged (but not compulsory)".

  12. jonatack commented at 11:29 AM on November 4, 2019: member

    utACK 690684dd1b0bce1c9190b6300f716b04d6496e1c modulo non-compulsory nit. I read but did not test this.

  13. michaelfolkson force-pushed on Nov 5, 2019
  14. in .github/ISSUE_TEMPLATE/good_first_issue.md:20 in 2d6cf7ef1f outdated
      15 | +
      16 | +Want to work on this issue?
      17 | +
      18 | +You do not need to request permission to start working on this. You are encouraged to comment on the issue if you are planning to work on it. This will help other contributors monitor which issues are actively being addressed and is also an effective way to request assistance if and when you need it.
      19 | +
      20 | +For guidance on contributing, please read [CONTRIBUTING.md](https://github.com/bitcoin/bitcoin/blob/master/CONTRIBUTING.md) before opening your pull request. 
    


    MarcoFalke commented at 4:37 PM on November 5, 2019:
    For guidance on contributing, please read [CONTRIBUTING.md](https://github.com/bitcoin/bitcoin/blob/master/CONTRIBUTING.md) before opening your pull request.
    
  15. MarcoFalke commented at 4:38 PM on November 5, 2019: member
  16. doc: Add template for good first issues 7b78b8d3a6
  17. michaelfolkson force-pushed on Nov 5, 2019
  18. MarcoFalke commented at 5:42 PM on November 5, 2019: member

    ACK 7b78b8d3a64503d582af8298241a20ebf582a0c5

  19. jonatack commented at 5:52 PM on November 5, 2019: member

    ACK 7b78b8d3a64503d582af8298241a20ebf582a0c5

  20. laanwj referenced this in commit 86771d4310 on Nov 6, 2019
  21. laanwj merged this on Nov 6, 2019
  22. laanwj closed this on Nov 6, 2019

  23. sidhujag referenced this in commit 3312088fb2 on Nov 7, 2019
  24. sidhujag referenced this in commit 471e5a2b8d on Nov 10, 2020
  25. PastaPastaPasta referenced this in commit c814c07fd1 on Jun 27, 2021
  26. PastaPastaPasta referenced this in commit 8e06bcb862 on Jun 28, 2021
  27. PastaPastaPasta referenced this in commit 550fffa10b on Jun 29, 2021
  28. PastaPastaPasta referenced this in commit 4ef9dbdd3d on Jul 1, 2021
  29. PastaPastaPasta referenced this in commit 7fc0b66336 on Jul 1, 2021
  30. PastaPastaPasta referenced this in commit 1fc1028297 on Jul 14, 2021
  31. MarcoFalke locked this on Dec 16, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-17 06:14 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me