[Github] Fix default PR description template #6633
Conversation
There was a problem hiding this comment.
Why not:
| Q | A
| --------------- | ---
| Bug fix? | no\|yes
| New feature? | no\|yes
| BC breaks? | no\|yes
| Related tickets | fixes #X, partially #Y, mentioned in #Z
| License | MIT| Q | A |
|---|---|
| Bug fix? | no |
| New feature? | no |
| BC breaks? | no |
| Related tickets | fixes #X, partially #Y, mentioned in #Z |
| License | MIT |
Zales0123
force-pushed
the
fix-default-pr-description-template
branch
from
18dc753
to
82bdc4e
Compare
|
👍 |
|
We can remove |
Zales0123
force-pushed
the
fix-default-pr-description-template
branch
from
82bdc4e
to
2e096d2
Compare
|
Ups 😄 |
| | BC breaks? | no|yes | ||
| | Bug fix? | no\|yes | ||
| | New feature? | no\|yes | ||
| | BC breaks? | no\|yes |
There was a problem hiding this comment.
How about | BC breaks? | no/yes? 😄
There was a problem hiding this comment.
@Zales0123 Please adjust the template and it is good to go!
There was a problem hiding this comment.
I've done this way, as @pamil suggested previously, to keep this | signs rather than replace them into something else. So I don't know now, should it be | or / 😄
There was a problem hiding this comment.
But why? I want to choose "yes" or "no" and that's it, so using "/" is one less backspace.
There was a problem hiding this comment.
Sure, but the point was not to change how PR table looks like, but only to fix it :) I have no problem with slash and could change it in a minute ;)
Zales0123
force-pushed
the
fix-default-pr-description-template
branch
from
2e096d2
to
f4c79dd
Compare
|
👍 |
There were problems with
|signs in few rows.Previous:
Now:
