Upgrade ember-cli-addon-docs and friends

* Add a dependency on ember-data since it no longer brings along the depenency. * Add newlines to account for bugs in ember-cli-addon-docs. See: ember-learn/ember-cli-addon-docs#473 and ember-learn/ember-cli-addon-docs#402 * Add a missing `END-SNIPPET` without which parsing of the snippet was corrupted
Addepar · Feb 5, 2021 · 701ece1 · 701ece1
1 parent 6c7accb
commit 701ece1
Show file tree

Hide file tree

Showing 7 changed files with 2,884 additions and 1,109 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -46,7 +46,9 @@ jobs:
             ember-lts-2.18,
             ember-lts-3.4,
             ember-lts-3.8,
+            ember-lts-3.12,
             ember-lts-3.16,
+            ember-lts-3.20,
             ember-release,
             ember-beta,
             ember-canary,

diff --git a/config/ember-try.js b/config/ember-try.js
@@ -36,7 +36,9 @@ module.exports = function() {
             devDependencies: {
               'ember-source': null,
               'ember-angle-bracket-invocation-polyfill': null,
+              'ember-data': null,
               'ember-cli-addon-docs': null,
+              'ember-cli-addon-docs-yuidoc': null,
             },
           },
         },
@@ -45,7 +47,9 @@ module.exports = function() {
           npm: {
             devDependencies: {
               'ember-source': '~2.12.0',
+              'ember-data': null,
               'ember-cli-addon-docs': null,
+              'ember-cli-addon-docs-yuidoc': null,
             },
           },
         },
@@ -54,7 +58,9 @@ module.exports = function() {
           npm: {
             devDependencies: {
               'ember-source': '~2.18.0',
+              'ember-data': null,
               'ember-cli-addon-docs': null,
+              'ember-cli-addon-docs-yuidoc': null,
             },
           },
         },
@@ -63,7 +69,9 @@ module.exports = function() {
           npm: {
             devDependencies: {
               'ember-source': '~3.4.0',
+              'ember-data': null,
               'ember-cli-addon-docs': null,
+              'ember-cli-addon-docs-yuidoc': null,
             },
           },
         },
@@ -72,7 +80,20 @@ module.exports = function() {
           npm: {
             devDependencies: {
               'ember-source': '~3.8.0',
+              'ember-data': null,
               'ember-cli-addon-docs': null,
+              'ember-cli-addon-docs-yuidoc': null,
+            },
+          },
+        },
+        {
+          name: 'ember-lts-3.12',
+          npm: {
+            devDependencies: {
+              'ember-source': '~3.12.0',
+              'ember-data': null,
+              'ember-cli-addon-docs': null,
+              'ember-cli-addon-docs-yuidoc': null,
             },
           },
         },
@@ -81,7 +102,20 @@ module.exports = function() {
           npm: {
             devDependencies: {
               'ember-source': '~3.16.0',
+              'ember-data': null,
+              'ember-cli-addon-docs': null,
+              'ember-cli-addon-docs-yuidoc': null,
+            },
+          },
+        },
+        {
+          name: 'ember-lts-3.20',
+          npm: {
+            devDependencies: {
+              'ember-source': '~3.18.0',
+              'ember-data': null,
               'ember-cli-addon-docs': null,
+              'ember-cli-addon-docs-yuidoc': null,
             },
           },
         },
@@ -90,7 +124,9 @@ module.exports = function() {
           npm: {
             devDependencies: {
               'ember-source': urls[0],
+              'ember-data': null,
               'ember-cli-addon-docs': null,
+              'ember-cli-addon-docs-yuidoc': null,
             },
           },
         },
@@ -99,7 +135,9 @@ module.exports = function() {
           npm: {
             devDependencies: {
               'ember-source': urls[1],
+              'ember-data': null,
               'ember-cli-addon-docs': null,
+              'ember-cli-addon-docs-yuidoc': null,
             },
           },
         },
@@ -108,7 +146,9 @@ module.exports = function() {
           npm: {
             devDependencies: {
               'ember-source': urls[2],
+              'ember-data': null,
               'ember-cli-addon-docs': null,
+              'ember-cli-addon-docs-yuidoc': null,
             },
           },
         },

diff --git a/package.json b/package.json
@@ -58,8 +58,8 @@
     "ember-a11y-testing": "^0.5.0",
     "ember-angle-bracket-invocation-polyfill": "^1.3.1",
     "ember-cli": "~3.1.4",
-    "ember-cli-addon-docs": "0.6.13",
-    "ember-cli-addon-docs-yuidoc": "^0.2.1",
+    "ember-cli-addon-docs": "^1.0.0",
+    "ember-cli-addon-docs-yuidoc": "^1.0.0",
     "ember-cli-dependency-checker": "^3.2.0",
     "ember-cli-deploy": "^1.0.2",
     "ember-cli-deploy-build": "^1.1.1",
@@ -70,6 +70,7 @@
     "ember-cli-shims": "^1.2.0",
     "ember-cli-sri": "^2.1.0",
     "ember-cli-uglify": "^2.0.0",
+    "ember-data": "~3.24.0",
     "ember-disable-prototype-extensions": "^1.1.2",
     "ember-export-application-global": "^2.0.0",
     "ember-faker": "^1.5.0",
@@ -80,7 +81,7 @@
     "ember-qunit": "^4.5.1",
     "ember-radio-button": "^1.2.3",
     "ember-resolver": "^5.1.1",
-    "ember-source": "~3.12",
+    "ember-source": "~3.24.0",
     "ember-source-channel-url": "^1.0.1",
     "ember-truth-helpers": "^2.0.0",
     "ember-try": "^1.1.0",

diff --git a/tests/dummy/app/pods/docs/guides/header/fixed-columns/controller.js b/tests/dummy/app/pods/docs/guides/header/fixed-columns/controller.js
@@ -62,6 +62,7 @@ export default Controller.extend({
       { name: 'K', valuePath: 'K' },
     ]);
   }),
+  // END-SNIPPET
 
   actions: {
     toggleFixed(column) {

diff --git a/tests/dummy/app/pods/docs/guides/header/scroll-indicators/template.md b/tests/dummy/app/pods/docs/guides/header/scroll-indicators/template.md
@@ -9,6 +9,7 @@ respective direction.
   {{#demo.example name='scroll-indicators'}}
     <div class="demo-container">
       {{! BEGIN-SNIPPET docs-example-scroll-indicators.hbs }}
+
         <EmberTable as |t|>
           <t.head
             @columns={{columns}}
@@ -33,6 +34,7 @@ them when they are present, or at the edges of the table when they are not.
   {{#demo.example name='scroll-indicators-with-fixed'}}
     <div class="demo-container">
       {{! BEGIN-SNIPPET docs-example-scroll-indicators-with-fixed.hbs }}
+
         <EmberTable as |t|>
           <t.head
             @columns={{columnsWithFixed}}
@@ -47,6 +49,7 @@ them when they are present, or at the edges of the table when they are not.
   {{demo.snippet name='docs-example-scroll-indicators-with-fixed.hbs'}}
   {{demo.snippet name='docs-example-scroll-indicators-with-fixed.js' label='component.js'}}
 {{/docs-demo}}
+
 ## Vertical Scroll Indicators with a Header & Footer
 
 Vertical scroll indicators respect both headers and footers, appearing just
@@ -56,6 +59,7 @@ inside any sticky rows at the top or bottom of the table.
   {{#demo.example name='scroll-indicators-with-footer'}}
     <div class="demo-container">
       {{! BEGIN-SNIPPET docs-example-scroll-indicators-with-footer.hbs }}
+
         <EmberTable as |t|>
           <t.head
             @columns={{columnsWithFooter}}

diff --git a/tests/dummy/app/pods/docs/guides/header/sorting/template.md b/tests/dummy/app/pods/docs/guides/header/sorting/template.md
@@ -66,7 +66,11 @@ you want to sort the content of the table asynchronously, you can unset the
 `sortFunction` and handle the async request yourself.
 
 ```hbs
-{{ember-thead sortFunction=null}}
+<EmberTable as |t|>
+  <t.head
+    sortFunction=null
+  />
+</EmberTable>
 ```
 
 ## Disabling Sorting