Skip to content

Commit d671a64

Browse files
committed
More upgrades
1 parent 0080810 commit d671a64

32 files changed

+24195
-2132
lines changed

.eslintignore

+20
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
1+
# unconventional js
2+
/blueprints/*/files/
3+
/vendor/
4+
5+
# compiled output
6+
/dist/
7+
/tmp/
8+
9+
# dependencies
10+
/bower_components/
11+
/node_modules/
12+
13+
# misc
14+
/coverage/
15+
!.*
16+
17+
# ember-try
18+
/.node_modules.ember-try/
19+
/bower.json.ember-try
20+
/package.json.ember-try

.eslintrc.js

+30-2
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,38 @@ module.exports = {
44
ecmaVersion: 2017,
55
sourceType: 'module'
66
},
7-
extends: 'eslint:recommended',
7+
plugins: [
8+
'ember'
9+
],
10+
extends: [
11+
'eslint:recommended',
12+
'plugin:ember/recommended'
13+
],
814
env: {
915
browser: true
1016
},
1117
rules: {
12-
}
18+
},
19+
overrides: [
20+
// node files
21+
{
22+
files: [
23+
'.eslintrc.js',
24+
'.template-lintrc.js',
25+
'ember-cli-build.js',
26+
'testem.js',
27+
'blueprints/*/index.js',
28+
'config/**/*.js',
29+
'lib/*/index.js'
30+
],
31+
parserOptions: {
32+
sourceType: 'script',
33+
ecmaVersion: 2015
34+
},
35+
env: {
36+
browser: false,
37+
node: true
38+
}
39+
}
40+
]
1341
};

.gitignore

+12-11
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,24 @@
11
# See https://help.github.com/ignore-files/ for more about ignoring files.
22

33
# compiled output
4-
/dist
5-
/tmp
4+
/dist/
5+
/tmp/
66

77
# dependencies
8-
/node_modules
9-
/bower_components
8+
/bower_components/
9+
/node_modules/
1010

1111
# misc
12+
/.env*
1213
/.sass-cache
1314
/connect.lock
14-
/coverage/*
15+
/coverage/
1516
/libpeerconnection.log
16-
npm-debug.log*
17-
yarn-error.log
18-
testem.log
17+
/npm-debug.log*
18+
/testem.log
19+
/yarn-error.log
1920

2021
# ember-try
21-
.node_modules.ember-try/
22-
bower.json.ember-try
23-
package.json.ember-try
22+
/.node_modules.ember-try/
23+
/bower.json.ember-try
24+
/package.json.ember-try

.template-lintrc.js

+5
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
'use strict';
2+
3+
module.exports = {
4+
extends: 'recommended'
5+
};

.travis.yml

+2
Original file line numberDiff line numberDiff line change
@@ -25,4 +25,6 @@ install:
2525
- yarn install --non-interactive
2626

2727
script:
28+
- yarn lint:hbs
29+
- yarn lint:js
2830
- yarn test

app/components/basic-tree.js

+18-12
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,29 @@
11
import Ember from 'ember';
22

3-
// Import the D3 packages we want to use
4-
import { select, event } from 'd3-selection';
5-
import { cluster, hierarchy } from 'd3-hierarchy';
6-
import { zoom, zoomIdentity } from 'd3-zoom';
7-
8-
const { run, get, inject } = Ember;
3+
const {
4+
select,
5+
hierarchy,
6+
cluster,
7+
zoom,
8+
zoomIdentity
9+
} = self.d3;
10+
11+
const { inject } = Ember;
12+
import { run } from '@ember/runloop';
13+
import { get } from '@ember/object';
14+
import Component from '@ember/component';
915

1016
const DURATION = 500;
1117

1218
// The offset amount (in px) from the left or right side of a node
1319
// box to offset lines between nodes, so the lines don't come right
1420
// up to the edge of the box.
15-
const NODE_OFFSET_SIZE = 50;
21+
const NODE_OFFSET_SIZE = 50;
1622

1723
// copied these functions temporarily from `broccoli-viz` here:
1824
// https://github.com/ember-cli/broccoli-viz/blob/master/lib/node-by-id.js
1925

20-
export default Ember.Component.extend({
26+
export default Component.extend({
2127
classNames: ['basic-tree'],
2228

2329
graph: inject.service(),
@@ -120,7 +126,7 @@ export default Ember.Component.extend({
120126
// for debugging
121127
self.root = root;
122128

123-
// Create the graph. The nodeSize() is [8,280] (width, height) because we
129+
// Create the graph. The nodeSize() is [8,280] (width, height) because we
124130
// want to change the orientation of the graph from top-down to left-right.
125131
// To do that we reverse X and Y for calculations and translations.
126132
let graph = cluster()
@@ -177,7 +183,7 @@ export default Ember.Component.extend({
177183
.style('fill', "#fff");
178184

179185
// Draw a box in a separate color for the first line as
180-
// a 'title'.
186+
// a 'title'.
181187
nodeEnter.append("rect")
182188
.attr('x', 0)
183189
.attr('y', '-2em')
@@ -258,7 +264,7 @@ export default Ember.Component.extend({
258264
.attr("d", function(d) {
259265
let sourceExitY = d.source.y + d.source.computedWidth + NODE_OFFSET_SIZE;
260266
let targetEntranceY = d.target.y - NODE_OFFSET_SIZE;
261-
267+
262268
return "M" + d.target.y + "," + d.target.x
263269
+ "L" + targetEntranceY + "," + d.target.x
264270
+ " " + sourceExitY + "," + d.target.x
@@ -272,7 +278,7 @@ export default Ember.Component.extend({
272278
.attr("d", function(d) {
273279
let sourceExitY = d.source.y + d.source.computedWidth + NODE_OFFSET_SIZE;
274280
let targetEntranceY = d.target.y - NODE_OFFSET_SIZE;
275-
281+
276282
return "M" + d.target.y + "," + d.target.x
277283
+ "L" + targetEntranceY + "," + d.target.x
278284
+ " " + sourceExitY + "," + d.target.x

app/components/flame-graph.js

+8-3
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,18 @@
11
import Ember from 'ember';
22
import FlameGraph from '../utils/d3-flame-graphs-v4/d3-flame-graph';
33

4-
const { run, get, inject } = Ember;
4+
import { run } from '@ember/runloop';
5+
import { get } from '@ember/object';
6+
import Component from '@ember/component';
7+
import { readOnly } from '@ember/object/computed';
58

6-
export default Ember.Component.extend({
9+
const { inject } = Ember;
10+
11+
export default Component.extend({
712
classNames: ['flame-graph'],
813
graph: inject.service(),
914
flameGraph: null,
10-
totalTime: Ember.computed.alias('graph.data.summary.totalTime'),
15+
totalTime: readOnly('graph.data.summary.totalTime'),
1116

1217
init() {
1318
this._super(...arguments);

app/components/slow-node-times.js

+7-7
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
11
import Ember from 'ember';
2+
import { get, set } from '@ember/object';
3+
import { computed } from '@ember/object';
4+
import Component from '@ember/component';
25

36
const {
4-
get,
5-
set,
6-
computed,
77
inject
88
} = Ember;
99

@@ -32,13 +32,13 @@ function computeNodeTimes(node) {
3232
}
3333
}
3434

35-
Ember.set(node._stats.time, 'plugin', total);
35+
set(node._stats.time, 'plugin', total);
3636

3737
return total;
3838
}
3939

4040

41-
export default Ember.Component.extend({
41+
export default Component.extend({
4242
graph: inject.service(),
4343

4444
init() {
@@ -85,7 +85,7 @@ export default Ember.Component.extend({
8585
}, {});
8686

8787
nodes = [];
88-
88+
8989
for (let pluginName in pluginNameMap) {
9090
nodes.push({
9191
groupedByPluginName: true,
@@ -112,7 +112,7 @@ export default Ember.Component.extend({
112112
// off the label as the plugin name. If not, we need
113113
// to create a map of the plugin names and return that.
114114
let pluginNames = [];
115-
115+
116116
if (nodes[0].groupedByPluginName === true) {
117117
pluginNames = nodes.map(node => node.label.name);
118118
} else {

app/controllers/application.js

+2-1
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,12 @@
11
import Ember from 'ember';
22
import fetch from "fetch";
3+
import Controller from '@ember/controller';
34

45
const {
56
inject
67
} = Ember;
78

8-
export default Ember.Controller.extend({
9+
export default Controller.extend({
910
graph: inject.service(),
1011

1112
actions: {

app/controllers/flame.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import Ember from 'ember';
2+
import Controller from '@ember/controller';
23

34
const {
4-
Controller,
55
inject
66
} = Ember;
77

app/controllers/graph.js

+3-3
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
11
import Ember from 'ember';
2+
import Controller from '@ember/controller';
3+
import { getOwner } from '@ember/application';
4+
import { computed } from '@ember/object';
25

36
const {
4-
Controller,
5-
getOwner,
6-
computed,
77
inject
88
} = Ember;
99

app/controllers/graph/index.js

+2-1
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,10 @@
11
import Ember from 'ember';
2+
import Controller from '@ember/controller';
23

34
const {
45
inject
56
} = Ember;
67

7-
export default Ember.Controller.extend({
8+
export default Controller.extend({
89
graph: inject.service()
910
});

app/controllers/graph/node.js

+2-1
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,10 @@
1+
import Controller from '@ember/controller';
12
import Ember from 'ember';
23

34
const {
45
inject
56
} = Ember;
67

7-
export default Ember.Controller.extend({
8+
export default Controller.extend({
89
graph: inject.service()
910
});

app/controllers/selected-node.js

+2-1
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,10 @@
11
import Ember from 'ember';
2+
import Controller from '@ember/controller';
23

34
const {
45
inject
56
} = Ember;
67

7-
export default Ember.Controller.extend({
8+
export default Controller.extend({
89
graph: inject.service()
910
})

app/controllers/slow-nodes.js

+2-1
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,11 @@
1+
import Controller from '@ember/controller';
12
import Ember from 'ember';
23

34
const {
45
inject
56
} = Ember;
67

7-
export default Ember.Controller.extend({
8+
export default Controller.extend({
89
graph: inject.service(),
910

1011
actions: {

app/helpers/includes.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
1-
import Ember from 'ember';
1+
import { helper as buildHelper } from '@ember/component/helper';
22

3-
export default Ember.Helper.helper(function([haystack, needle]) {
3+
export default buildHelper(function([haystack, needle]) {
44
return haystack && haystack.includes && haystack.includes(needle);
55
});

app/helpers/ns-to-ms.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
1-
import Ember from 'ember';
1+
import { helper as buildHelper } from '@ember/component/helper';
22

33
export function nsToMs([time]) {
44
return (time / 1000000).toFixed(2);
55
}
66

7-
export default Ember.Helper.helper(nsToMs);
7+
export default buildHelper(nsToMs);

app/helpers/stats-iterator.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
1-
import Ember from 'ember';
1+
import { helper as buildHelper } from '@ember/component/helper';
22

3-
export default Ember.Helper.helper(function([node]) {
3+
export default buildHelper(function([node]) {
44
let stats = {};
55

66
for (let [name, value] of node.statsIterator()) {

app/services/graph.js

+3-6
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,12 @@
1-
import Ember from 'ember';
21
import config from '../config/environment';
32
import heimdallGraph from 'heimdalljs-graph';
4-
5-
const {
6-
getOwner
7-
} = Ember;
3+
import { getOwner } from '@ember/application';
4+
import Service from '@ember/service';
85

96
const DATA_STORAGE_KEY = `${config.storageVersion}_graph-data`;
107
const SELECTED_NODE_STORAGE_KEY = `${config.storageVersion}_selected-node-id`;
118

12-
export default Ember.Service.extend({
9+
export default Service.extend({
1310
init() {
1411
this._super(...arguments);
1512

0 commit comments

Comments
 (0)