Skip to content
This repository was archived by the owner on Apr 27, 2023. It is now read-only.

Commit f232841

Browse files
authored
build: lint sourceCode (#12)
1 parent edb985b commit f232841

File tree

3 files changed

+11
-14
lines changed

3 files changed

+11
-14
lines changed

.github/workflows/ci.yml

Lines changed: 1 addition & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -83,18 +83,6 @@ jobs:
8383
md5sum ${{ steps.metadata.outputs.archive }} > ${{ steps.metadata.outputs.archive-checksum }}
8484
echo "::set-output name=checksum::$(cat ./${{ steps.metadata.outputs.archive-checksum }} | cut -d' ' -f1)"
8585
86-
#
87-
# analyzers:
88-
# screenshots:
89-
# rules:
90-
# screenshots:
91-
# # TODO publish screenshots?
92-
# enabled: false
93-
#
94-
#analyzers:
95-
# version:
96-
# enabled: false
97-
9886
9987
# A PR created by pyroscopebot (aka Release Please)
10088
# bumps package.json and CHANGELOG, which we then want to validate
@@ -115,7 +103,7 @@ jobs:
115103
pushd ./plugin-validator/pkg/cmd/plugincheck2
116104
go install
117105
popd
118-
plugincheck2 -config lint.config.yaml -sourceCodeUri=. ${{ steps.metadata.outputs.archive }}
106+
plugincheck2 -config lint.config.yaml -sourceCodeUri=file://./ ${{ steps.metadata.outputs.archive }}
119107
env:
120108
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
121109

.github/workflows/release.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,8 @@ jobs:
101101
pushd ./plugin-validator/pkg/cmd/plugincheck2
102102
go install
103103
popd
104-
plugincheck2 -config lint.config.yaml -sourceCodeUri=. ${{ steps.metadata.outputs.archive }}
104+
plugincheck2 -config lint.config.yaml -sourceCodeUri=file://./ ${{ steps.metadata.outputs.archive }}
105+
105106
# until here it's pretty much the same as ci.yml
106107
# TODO: share the code somehow
107108

jest.config.js

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
// This file is needed because it is used by vscode and other tools that
2+
// call `jest` directly. However, unless you are doing anything special
3+
// do not edit this file
4+
5+
const standard = require('@grafana/toolkit/src/config/jest.plugin.config');
6+
7+
// This process will use the same config that `yarn test` is using
8+
module.exports = standard.jestConfig();

0 commit comments

Comments
 (0)