Skip to content

Add global issue templates#53

Merged
greg0ire merged 1 commit intodoctrine:mainfrom
greg0ire:pr-template
Oct 19, 2024
Merged

Add global issue templates#53
greg0ire merged 1 commit intodoctrine:mainfrom
greg0ire:pr-template

Conversation

@greg0ire
Copy link
Copy Markdown
Member

We also have PR templates at least in ORM, but I do not think they are used at all, since you have to add a ?template=the_template.md parameter to enable them.


We try to keep Github issues for bug reports and feature requests only. If
you have a question, please ask it on Stack Overflow or in one of the channels
mentioned at https://www.doctrine-project.org/community/index.html
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ORM can keep its own template for this, that hints at github discussions instead.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ORM, for example, has Support Questions in GitHub Discussions. I don't know if those should be also enabled in other repositories, but it looks like those won't be configured with a markdown template afaik.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should mention the GitHub Discussions board on the linked page though.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@derrabus Other repositories don't have Discussions enabled. Are we going to enable them in every repository and mention them in this template, or do you suggest this for the ORM template?

Copy link
Copy Markdown
Member Author

@greg0ire greg0ire Oct 12, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think @derrabus means to link to the ORM Github Discussions from the community page.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Exactly.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll mention that you can configure issue templates and add your own types that link to an external resource, rather than opening the Github issue form. In the MongoDB extension config we specifcally use this to link to the community forums.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@alcaeus that's really good to know given that I'm stating that we should not use github issues for support, and then the community page I links back to Github… could be confusing! I'll try something similar to what you did.

Comment on lines +28 to +37

<!--
Provide a failing Unit or Functional Test - you can submit one in a Pull
Request separately, referencing this bug report.
If that is too difficult, provide a link to a minimal repository containing an
application that reproduces the bug.
If the bug is simple, you may provide a code snippet instead, or even a list of
steps.
-->
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A full pull request with a fix would be welcome too. Some people still might not know that they can already provide a fix for a bug that they found in Doctrine code and just need a nudge into that direction.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've added a short sentence to add that nudge. What do you think?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like it.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this rather for non-technical description of a new feature? Otherwise I would also suggest something like an expected behaviour on how the creator of the feature request believes how this feature could be implemented.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've reused a template we use in my company, with 3 sections:

  • Why
  • What
  • How

All 3 sections have comments asking for something.

Does that work for you?


We try to keep Github issues for bug reports and feature requests only. If
you have a question, please ask it on Stack Overflow or in one of the channels
mentioned at https://www.doctrine-project.org/community/index.html
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should mention the GitHub Discussions board on the linked page though.

Comment on lines +8 to +18
#### Why

<!--
Explain why you believe this feature should be implemented.
-->

#### What

<!--
Describe the behavior you would like to see.
-->
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suggest flipping the what and why; often the "why" becomes quite apparent when describing the suggested feature. I'm also blindly assuming that most such requests come from "I want to use feature X that isn't supported" thoughts, rather than "I have problem X that needs to be solved by a new feature". In that case, the thought process would always lead someone to a "what" before going on to the "why".

Suggested change
#### Why
<!--
Explain why you believe this feature should be implemented.
-->
#### What
<!--
Describe the behavior you would like to see.
-->
#### What
<!--
Describe the behavior you would like to see.
-->
#### Why
<!--
Explain why you believe this feature should be implemented.
-->


We try to keep Github issues for bug reports and feature requests only. If
you have a question, please ask it on Stack Overflow or in one of the channels
mentioned at https://www.doctrine-project.org/community/index.html
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll mention that you can configure issue templates and add your own types that link to an external resource, rather than opening the Github issue form. In the MongoDB extension config we specifcally use this to link to the community forums.

We also have PR templates at least in ORM, but I do not think they are
used at all, since you have to add a `?template=the_template.md`
parameter to enable them.
@greg0ire greg0ire requested a review from alcaeus October 14, 2024 19:18
@greg0ire greg0ire merged commit b48df76 into doctrine:main Oct 19, 2024
@greg0ire greg0ire deleted the pr-template branch October 19, 2024 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants