Merge branch 'master' into troubleshooting-guide

Author: sanscontext

.github/PULL_REQUEST_TEMPLATE.md

@@ -4,8 +4,13 @@
 
 <!--Tell us what you did and why-->
 
+### Merge timing
+<!-- When should this get merged/published?
+- ASAP once approved?
+- On a specific date?
+- Depending on a specific project?-->
 
 ### Related issues (optional)
 
-<!--Refer to related PRs or issues: #1234, or 'Fixes #1234' or 'Closes #1234'.
-    Or link to full URLs to issues or pull requests in other Github projects -->
+<!--Refer to related PRs or issues: #1234 or 'Closes #1234'.
+    Or paste full URLs to issues or pull requests in other Github projects -->

.vale.ini

@@ -0,0 +1,6 @@
+StylesPath = vale-styles
+
+Vocab = Docs
+
+[*.md]
+BasedOnStyles = write-good, Segment

CODEOWNERS

@@ -1,13 +1,16 @@
 # The default owners for everything in
 # the repo. Unless a later match takes precedence.
-CODEOWNERS  @sanscontext @XavierAgostini @danielstjules @stephment
+CODEOWNERS  @segmentio/segment-doc-team
 
 # Diagram library
 /diagram-library @sanscontext
 
 # Utility scripts
-/scripts @sanscontext @XavierAgostini
+/scripts @segmentio/segment-doc-team @XavierAgostini
 
+# Vale Linting
+/vale-styles @segmentio/segment-doc-team
+.vale.ini @segmentio/segment-doc-team
 
 # Content owners should be in the order of PM, TL (team-lead), and EM (in a crisis) for a given team.
 # This team will receive review requests automatically when a PR is submitted modifying the files in
@@ -18,16 +21,14 @@ CODEOWNERS  @sanscontext @XavierAgostini @danielstjules @stephment
 
 
 # Libraries owners
-/src/connections/catalog/libraries  @osamakhn @bsneed @lubird
-
+/src/connections/catalog/libraries  @osamakhn @bsneed @pooyaj @juliofarah
 
 # Destination owners
-/src/connections/destinations   @n2parko @danielstjules @benhorowitz
-# temporary removal: @carlosmecha
+/src/connections/destinations  @segmentio/segment-doc-team
 
 
 # Privacy owners
-/src/privacy    @notfelineit @aliya-dossa
+/src/privacy    @notfelineit
 
 
 # Protocols owners

Gemfile.lock

@@ -1,18 +1,18 @@
 GIT
   remote: https://github.com/jekyll/jekyll.git
-  revision: 1d7d11be61196b45b4645fd8cd73776e8fe8d184
+  revision: e739c777fc8fb63b9890c0738983af6f1d525966
   specs:
-    jekyll (4.0.0)
+    jekyll (4.1.1)
       addressable (~> 2.4)
       colorator (~> 1.0)
       em-websocket (~> 0.5)
-      i18n (>= 0.9.5, < 2)
+      i18n (~> 1.0)
       jekyll-sass-converter (~> 2.0)
       jekyll-watch (~> 2.0)
-      kramdown (~> 2.1)
+      kramdown (~> 2.3)
       kramdown-parser-gfm (~> 1.0)
       liquid (~> 4.0)
-      mercenary (~> 0.3.3)
+      mercenary (~> 0.4.0)
       pathutil (~> 0.9)
       rouge (~> 3.0)
       safe_yaml (~> 1.0)
@@ -24,71 +24,73 @@ GEM
     addressable (2.7.0)
       public_suffix (>= 2.0.2, < 5.0)
     colorator (1.1.0)
-    commonmarker (0.20.1)
+    commonmarker (0.21.0)
       ruby-enum (~> 0.5)
-    concurrent-ruby (1.1.5)
-    dotenv (2.7.5)
-    em-websocket (0.5.1)
+    concurrent-ruby (1.1.7)
+    dotenv (2.7.6)
+    em-websocket (0.5.2)
       eventmachine (>= 0.12.9)
       http_parser.rb (~> 0.6.0)
     eventmachine (1.2.7)
-    faraday (0.17.1)
+    faraday (1.0.1)
       multipart-post (>= 1.2, < 3)
-    ffi (1.11.3)
+    ffi (1.13.1)
     forwardable-extended (2.6.0)
     http_parser.rb (0.6.0)
-    i18n (1.7.0)
+    i18n (1.8.5)
       concurrent-ruby (~> 1.0)
     jekyll-commonmark (1.3.1)
       commonmarker (~> 0.14)
       jekyll (>= 3.7, < 5.0)
     jekyll-include-cache (0.2.0)
       jekyll (>= 3.7, < 5.0)
-    jekyll-last-modified-at (1.2.1)
+    jekyll-last-modified-at (1.3.0)
       jekyll (>= 3.7, < 5.0)
       posix-spawn (~> 0.3.9)
-    jekyll-redirect-from (0.15.0)
+    jekyll-redirect-from (0.16.0)
       jekyll (>= 3.3, < 5.0)
-    jekyll-sass-converter (2.0.1)
+    jekyll-sass-converter (2.1.0)
       sassc (> 2.0.1, < 3.0)
     jekyll-sitemap (1.4.0)
       jekyll (>= 3.7, < 5.0)
     jekyll-watch (2.2.1)
       listen (~> 3.0)
-    kramdown (2.1.0)
+    kramdown (2.3.0)
+      rexml
     kramdown-parser-gfm (1.1.0)
       kramdown (~> 2.0)
     liquid (4.0.3)
     liquid-c (4.0.0)
       liquid (>= 3.0.0)
-    listen (3.2.0)
+    listen (3.2.1)
       rb-fsevent (~> 0.10, >= 0.10.3)
       rb-inotify (~> 0.9, >= 0.9.10)
-    mercenary (0.3.6)
+    mercenary (0.4.0)
     multipart-post (2.1.1)
     pathutil (0.16.2)
       forwardable-extended (~> 2.6)
-    posix-spawn (0.3.13)
-    premonition (2.0.0)
-    public_suffix (4.0.1)
+    posix-spawn (0.3.15)
+    premonition (2.0.1)
+    public_suffix (4.0.6)
     rake (13.0.1)
-    rb-fsevent (0.10.3)
-    rb-inotify (0.10.0)
+    rb-fsevent (0.10.4)
+    rb-inotify (0.10.1)
       ffi (~> 1.0)
-    rouge (3.13.0)
-    ruby-enum (0.7.2)
+    rexml (3.2.4)
+    rouge (3.23.0)
+    ruby-enum (0.8.0)
       i18n
     safe_yaml (1.0.5)
-    sassc (2.2.1)
+    sassc (2.4.0)
       ffi (~> 1.9)
     terminal-table (1.8.0)
       unicode-display_width (~> 1.1, >= 1.1.1)
     thread_safe (0.3.6)
-    tzinfo (1.2.5)
+    tzinfo (1.2.7)
       thread_safe (~> 0.1)
-    tzinfo-data (1.2019.3)
+    tzinfo-data (1.2020.2)
       tzinfo (>= 1.0.0)
-    unicode-display_width (1.6.0)
+    unicode-display_width (1.7.0)
     wdm (0.1.1)
 
 PLATFORMS

README.md

@@ -53,7 +53,7 @@ See the [Contributor Guide](contributors.md) for more info.
 
 **At minimum you must open a PR so the docs team gets a notification. Do not merge directly to master.**
 
-- **Just fixing a typo**? -> No review needed, you can admin-merge your PR with our blessings.
+- **Just fixing a typo**? -> No review needed, but please label it FIX in the PR subject so we know not to worry. You can then admin-merge your PR with our blessings.
 
 - **Delta of <20 words or ~150 characters**? -> Yes, but a minor review. Open a PR and tag @sanscontext, who'll review mostly for formatting and copy issues.
 
@@ -98,7 +98,7 @@ Once the PR is merged, the docs site rebuilds and the changes are live!
 
 ### Draft PRs
 
-If you're doing a substantial change and you're going to want to spend a few weeks on it, use [Github's Draft PRs feature](https://help.github.com/en/articles/about-pull-requests#draft-pull-requests), or add `WIP` to the title of your PR. This lets us know to ignore the PR until you're ready.
+If you're doing a substantial change and you're going to want to spend a few weeks on it, use [Github's Draft PRs feature](https://help.github.com/en/articles/about-pull-requests#draft-pull-requests), or add `WIP` to the title of your PR. This lets us know to ignore the PR until you're ready (otherwise Laura will ping you weekly about it!).
 
 
 ## How the docs build works
@@ -134,7 +134,7 @@ The most interesting ones are:
 
 ### Images
 
-**All images should be saved locally! No linking to 3rd party-hosted images!** Images are published to our CDN from the build step.
+**Save all images locally! No linking to 3rd party-hosted images!** Images are published to our CDN from the build step, and this means they won't go missing if the hosting service dujour goes out of business.
 
 There are no _enforced_ naming conventions at this time. Files that start with an underscore are ignored by Jekyll. Anything you see with `asset` was dowloaded by a script to migrate it out of Contents.io.
 

_config.yml

@@ -32,7 +32,7 @@ defaults:
       integration_type: source
       layout: integration
   - scope:
-      path: "connections/warehouses/catalog"
+      path: "connections/storage/catalog"
     values:
       integration_type: warehouse
       landing: true