Skip to content

[1] PR & issue template fixes #1 #2

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
w1stler wants to merge 4 commits into
base: master
Choose a base branch
from
+30 −0
Open

[1] PR & issue template fixes #1 #2

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
97f69b5
[1] PR & issue template
Dec 4, 2019
f37543b
[1] PEBKAC
w1stler Dec 5, 2019
e55d2dd
[1] Remove rollback from PR template
w1stler Jan 8, 2020
dd5ac33
[1] Remove unit tests from template
w1stler Mar 19, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 16 additions & 0 deletions ISSUE_TEMPLATE.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
#### Subject of the issue:
Describe your issue here

#### Your environment:
Copy link

@OtisRed OtisRed Dec 5, 2019

Choose a reason for hiding this comment

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

Do we need this, while working in docker?

Copy link
Member Author

@w1stler w1stler Dec 11, 2019 •
edited
Loading

Choose a reason for hiding this comment

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

By environment I meant production / RC / local. I am not sure if deployed envs' configuration will differ.

* version/branch
* which browser and its version
Copy link
Member

@arturtamborski arturtamborski Dec 5, 2019

Choose a reason for hiding this comment

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

I don't think that there's much of use out of it, we can probably assume that every member is using a "recent enough" one.

Copy link
Member Author

@w1stler w1stler Dec 11, 2019

Choose a reason for hiding this comment

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

Ditto, I would leave it as a good practice.


#### Steps to reproduce:
Tell us how to reproduce this issue

#### Actual behaviour:
Tell us what happened

#### Expected behaviour:
Tell us what should happen

14 changes: 14 additions & 0 deletions PULL_REQUEST_TEMPLATE.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
#### Story / Bug id:
123 [link to the issue]

#### Description:
Describe changes made in your pull request here

#### Migrations:
N/A

#### New imports / dependencies:
N/A

#### What tests do I need to run to validate this change:
N/A
Copy link
Member

@arturtamborski arturtamborski Dec 5, 2019

Choose a reason for hiding this comment

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

👍 💯