aboutsummaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorChocobozzz <me@florianbigard.com>2021-06-25 14:55:25 +0200
committerChocobozzz <me@florianbigard.com>2021-06-25 14:55:25 +0200
commit4f97f58383c37d11a8f8ae3ef6e2b761307cb4d0 (patch)
tree81464f0a1d9e3753e1e7aeed125fc121eb7c5560
parent37a44fc915eef2140e22ceb96aba6b6eb2509007 (diff)
downloadPeerTube-4f97f58383c37d11a8f8ae3ef6e2b761307cb4d0.tar.gz
PeerTube-4f97f58383c37d11a8f8ae3ef6e2b761307cb4d0.tar.zst
PeerTube-4f97f58383c37d11a8f8ae3ef6e2b761307cb4d0.zip
Update PR template and contributing guide
-rw-r--r--.github/CONTRIBUTING.md14
-rw-r--r--.github/PULL_REQUEST_TEMPLATE.md18
2 files changed, 14 insertions, 18 deletions
diff --git a/.github/CONTRIBUTING.md b/.github/CONTRIBUTING.md
index a37159fa8..47129ea74 100644
--- a/.github/CONTRIBUTING.md
+++ b/.github/CONTRIBUTING.md
@@ -18,8 +18,10 @@ Interested in contributing? Awesome!
18 - [Server side](#server-side) 18 - [Server side](#server-side)
19 - [Client side](#client-side) 19 - [Client side](#client-side)
20 - [Client and server side](#client-and-server-side) 20 - [Client and server side](#client-and-server-side)
21 - [Testing the federation of PeerTube servers](#testing-the-federation-of-peertube-servers) 21 - [RTL layout](#rtl-layout)
22 - [Unit tests](#unit-tests) 22 - [Testing](#testing)
23 - [Unit tests](#unit-tests)
24 - [Testing the federation of PeerTube servers](#testing-the-federation-of-peertube-servers)
23 - [Emails](#emails) 25 - [Emails](#emails)
24- [Plugins & Themes](#plugins--themes) 26- [Plugins & Themes](#plugins--themes)
25 27
@@ -224,14 +226,16 @@ $ createdb -O peertube peertube_test{1,2,3}
224Build the application and flush the old tests data: 226Build the application and flush the old tests data:
225 227
226``` 228```
227$ npm run build -- --light 229$ npm run build
228$ npm run clean:server:test 230$ npm run clean:server:test
229``` 231```
230 232
231This will run 3 nodes: 233To run 3 nodes:
232 234
233``` 235```
234$ npm run play 236$ NODE_APP_INSTANCE=1 NODE_ENV=test npm start
237$ NODE_APP_INSTANCE=2 NODE_ENV=test npm start
238$ NODE_APP_INSTANCE=3 NODE_ENV=test npm start
235``` 239```
236 240
237Then you will get access to the three nodes at `http://localhost:900{1,2,3}` 241Then you will get access to the three nodes at `http://localhost:900{1,2,3}`
diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md
index a49db848f..3c96818a3 100644
--- a/.github/PULL_REQUEST_TEMPLATE.md
+++ b/.github/PULL_REQUEST_TEMPLATE.md
@@ -6,24 +6,16 @@
6 6
7<!-- If suggesting a new feature or change, please discuss it in an issue first --> 7<!-- If suggesting a new feature or change, please discuss it in an issue first -->
8<!-- If fixing a bug, there should be an issue describing it with steps to reproduce --> 8<!-- If fixing a bug, there should be an issue describing it with steps to reproduce -->
9<!-- start with closing keywords if any apply: https://docs.github.com/en/enterprise/2.16/user/github/managing-your-work-on-github/closing-issues-using-keywords -->
10 9
11## Has this been tested? 10## Has this been tested?
12 11
13<!--- Put an `x` in the box that applies: --> 12<!-- Put an `x` in the box that applies: -->
13<!-- Check the unit test guide: https://docs.joinpeertube.org/contribute-getting-started?id=unit-tests -->
14
14- [ ] 👍 yes, I added tests to the test suite 15- [ ] 👍 yes, I added tests to the test suite
15- [ ] 👍 yes, light tests as follows are enough
16- [ ] 💭 no, because this PR is a draft and still needs work 16- [ ] 💭 no, because this PR is a draft and still needs work
17- [ ] 🙅 no, because they aren't needed 17- [ ] 🙅 no, because this PR does not update server code
18- [ ] 🙋 no, because I need help 18- [ ] 🙋 no, because I need help <!-- Detail how we can help you >
19
20<!--
21If you didn't test via unit-testing, you will still be asked to prove your fix is effective, doesn't have side-effects, or that your feature simply works:
22
23Please describe the tests that you ran to verify your changes.
24Provide instructions so we can reproduce.
25Please also list any relevant details for your test configuration(s):
26-->
27 19
28## Screenshots 20## Screenshots
29 21