Skip to content

Commit

Permalink
Merge pull request #9412 from js-org/MattIPv4/pr-template
Browse files Browse the repository at this point in the history
Require link + explanation in PR template
  • Loading branch information
indus authored Nov 25, 2024
2 parents bfd836d + e0344e3 commit 89ce618
Show file tree
Hide file tree
Showing 3 changed files with 17 additions and 7 deletions.
19 changes: 17 additions & 2 deletions PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -1,2 +1,17 @@
- [x] There is reasonable content on the page (see: [No Content](https://github.com/js-org/js.org/wiki/No-Content))
- [x] I have read and accepted the [Terms and Conditions](http://js.org/terms.html)
<!--
Thanks for creating a pull request to request a new subdomain from JS.ORG
Before creating your pull request, please complete the following steps:
- Ensure that your pull request changes only the cnames_active.js file, adding a single new line for your subdomain request
- Tick the two checkboxes, agreeing to the sentences, below by placing an x inside the square brackets ([ ] becomes [x])
- Add a link (GitHub repository, Vercel deployment, etc.) and explanation below for your content so we can validate your request
-->

- [ ] There is reasonable content on the page (see: [No Content](https://github.com/js-org/js.org/wiki/No-Content))
- [ ] I have read and accepted the [Terms and Conditions](http://js.org/terms.html)
- The site content can be seen at <link>

> The site content is ...
2 changes: 0 additions & 2 deletions PULL_REQUEST_TEMPLATE/addExternalPage.md

This file was deleted.

3 changes: 0 additions & 3 deletions PULL_REQUEST_TEMPLATE/addGitHubPage.md

This file was deleted.

0 comments on commit 89ce618

Please sign in to comment.