Skip to content
This repository was archived by the owner on Mar 4, 2020. It is now read-only.

Commit f0c5ff7

Browse files
perf: Store perf bundle and results to CI artifacts (#2256)
* perf: store perf bundle and results to CI artifacts * do not store any artifacts in danger
1 parent 32a3d9b commit f0c5ff7

File tree

2 files changed

+2
-12
lines changed

2 files changed

+2
-12
lines changed

.circleci/config.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -105,8 +105,8 @@ jobs:
105105
yarn danger ci
106106
fi
107107
- store_artifacts:
108-
path: dist/artifacts
109-
destination: artifacts
108+
path: packages/perf-test/dist
109+
destination: artifacts/perf
110110
- run:
111111
name: Publish npm package (master only)
112112
command: |

build/dangerjs/checkPerfRegressions.ts

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
import * as _ from 'lodash'
2-
import * as fs from 'fs-extra'
32
import * as path from 'path'
43

54
import { DangerJS } from './types'
@@ -43,15 +42,6 @@ function fluentFabricComparision(danger, markdown, warn) {
4342
},
4443
)
4544

46-
fs.mkdirpSync(config.paths.ciArtifacts('perf'))
47-
48-
_.forEach(results, value => {
49-
fs.copyFileSync(
50-
value.fluentFlamegraphFile,
51-
config.paths.ciArtifacts('perf', path.basename(value.fluentFlamegraphFile)),
52-
)
53-
})
54-
5545
const getStatus = fluentToFabric =>
5646
fluentToFabric > 1 ? '🔧' : fluentToFabric >= 0.7 ? '🎯' : '🦄'
5747

0 commit comments

Comments
 (0)