From dd357f7b64145b1ac174dc491e90152a4a2db37e Mon Sep 17 00:00:00 2001 From: Jared Date: Fri, 28 Aug 2020 18:56:30 -0700 Subject: [PATCH] Update readme and PR template. --- .github/pull_request_template.md | 8 +++++++- README.md | 1 + 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md index acd64edba..507dd85f3 100644 --- a/.github/pull_request_template.md +++ b/.github/pull_request_template.md @@ -1,4 +1,10 @@ -*Add your PR comments here.* +**Important:** I realize Remotely is currently lacking in unit and integration tests. I was pretty newb when I wrote a lot of the foundational code, and I haven't had time to refactor and restructure. + +I'd like to start changing that, though. Enough people are starting to depend on Remotely that I want to make sure it's maintainable and lives on. So I need every PR to include automated tests that prove the new functionality is working. + +If you have the time and the desire, it would also be incredibly helpful if you included some new tests against services touched by your changes. I know that this is my mess you're cleaning up, so I thank you wholeheartedly for your help. The people who use Remotely appreciate it too. :) + +*Delete the above and add your PR comments here and delete the above.* --- diff --git a/README.md b/README.md index 54fb2024a..82c9a4b7c 100644 --- a/README.md +++ b/README.md @@ -2,6 +2,7 @@ A remote control and remote scripting solution, built with .NET Core, SignalR Core, and WebRTC. [![Build Status](https://dev.azure.com/translucency/Remotely/_apis/build/status/Remotely-ReleaseBuild?branchName=master)](https://dev.azure.com/translucency/Remotely/_build/latest?definitionId=17&branchName=master) +![GitHub Build](https://github.com/lucent-sea/Remotely/workflows/GitHub%20Build/badge.svg) ## Sponsor This Project Ongoing via GitHub: https://github.com/sponsors/lucent-sea