Merge remote-tracking branch 'upstream/master' into units2ascii

* upstream/master: (439 commits)
  [DOC] Auto-generate changelog entry for PR bids-standard#394
  Update 01-contributors.md
  [DOC] Auto-generate changelog entry for PR bids-standard#389
  [DOC] Auto-generate changelog entry for PR bids-standard#386
  moved list of extension proposals to the main BIDS website section 06-extensions remains, but now points to the main website
  Typo fix for typo fix
  Typo fix for typo fix
  [DOC] Auto-generate changelog entry for PR bids-standard#374
  [FIX] Fix some typos and prose style issues
  [DOC] Auto-generate changelog entry for PR bids-standard#383
  restructure according to @emdupre's proposal
  neurostars for everything not just technical
  Apply suggestions from code review
  MNT: Update Pipfile
  ENH: Append CSS instead of overriding it
  ENH: Add watermark to drafts
  [DOC] Auto-generate changelog entry for PR bids-standard#381
  Update src/06-extensions.md
  Retiring moderator duties for BEP021
  [DOC] Auto-generate changelog entry for PR bids-standard#380
  ...

.circleci/artifact_path

@@ -0,0 +1 @@
+0/home/circleci/project/site/01-introduction.html

.circleci/config.yml

@@ -1,6 +1,6 @@
 version: 2
 jobs:
-  build:
+  build_docs:
     docker:
       - image: circleci/python:3.6.6
     steps:
@@ -25,6 +25,23 @@ jobs:
           command: pipenv run mkdocs build --clean --strict --verbose
       - store_artifacts:
           path: site
+      - persist_to_workspace:
+          root: .
+          paths: site
+
+  linkchecker:
+    docker:
+      - image: yarikoptic/linkchecker:9.4.0.anchorfix1-1
+    steps:
+      - attach_workspace:
+          at: ~/build
+      - run:
+          command: |
+            chmod a+rX -R ~
+            linkchecker -t 1 ~/build/site/
+            # check external separately by pointing to all *html so no
+            # failures for local file:/// -- yoh found no better way,
+            linkchecker -t 1 --check-extern --ignore-url 'file:///.*' --ignore-url https://fonts.gstatic.com ~/build/site/*html ~/build/site/*/*.html
 
 # Auto changelog collector
   github-changelog-generator:
@@ -73,7 +90,7 @@ jobs:
           name: get remark styles
           command: |
            cd ~
-           npm install remark-cli@5.0.0 remark-lint@6.0.2 remark-preset-lint-recommended@3.0.2 remark-preset-lint-markdown-style-guide@2.1.2
+           cat ~/project/npm-requirements.txt | xargs npm install
       - run: # remark the auto generated changes.md
           name: remark on autogenerated CHANGES.md
           command: |
@@ -126,7 +143,10 @@ workflows:
   version: 2
   search_build:
     jobs:
-      - build
+      - build_docs
+      - linkchecker:
+          requires:
+            - build_docs
       - github-changelog-generator:
           filters:
                branches:

.github/pull_request_template.md

@@ -0,0 +1,25 @@
+--- PLEASE READ AND DELETE THE TEXT BELOW BEFORE OPENING THE PULL REQUEST ---
+
+See the [CONTRIBUTING](https://github.com/bids-standard/bids-specification/blob/master/CONTRIBUTING.md) guide. Specifically:
+
+- Please keep the title of your pull request (PR) short but informative - it will
+  appear in the changelog.
+- Please ensure your name is credited on our [Contributors appendix](https://github.com/bids-standard/bids-specification/blob/master/src/99-appendices/01-contributors.md).
+  To add your name, please edit our [Contributors wiki](https://github.com/bids-standard/bids-specification/wiki/Contributors) and add your name with the type of contribution.
+  For assistance, please contact @franklin-feingold or @sappelhoff.
+- Use one of the following prefixes in the title of your PR:
+  - `[ENH]` - enhancement of the specification that adds a new feature or
+    support for a new data type
+  - `[FIX]` - fix of a typo or language clarification
+  - `[INFRA]` - changes to the infrastructure automating the specification
+    release (for example building HTML docs etc.)
+  - `[MISC]` - everything else including changes to the file listing
+    contributors
+- If you are opening a PR to obtain early feedback, but the changes
+  are not ready to be merged (a.k.a. Work in Progress PR) please
+  use a [draft PR](https://github.blog/2019-02-14-introducing-draft-pull-requests/).
+- After opening the PR, our continuous integration services will automatically check your contribution  for formatting errors and render a preview of the BIDS specification with your changes.
+  To see the checks and preview, scroll down and click on the `show all checks` link.
+  From the list, select the `Details` link of the `ci/circleci: build_docs artifact` check to see the preview of the BIDS specification.
+
+--- PLEASE READ AND DELETE THE TEXT ABOVE BEFORE OPENING THE PULL REQUEST ---

.remarkrc

@@ -4,6 +4,7 @@
     ["lint-no-duplicate-headings", false],
     ["lint-list-item-indent", "tab-size"],
     ["lint-emphasis-marker", "consistent"],
-    ["lint-maximum-line-length", false]
+    ["lint-maximum-line-length", false],
+    ["lint-maximum-heading-length", false]
   ]
 }

.travis.yml

@@ -5,6 +5,6 @@ cache:
   directories:
   - node_modules # NPM packages
 before_script:
-  - npm install remark-cli@5.0.0 remark-lint@6.0.2 remark-preset-lint-recommended@3.0.2 remark-preset-lint-markdown-style-guide@2.1.2
+  - npm install `cat npm-requirements.txt`
 script:
   - remark src/*.md src/*/*.md --frail