Skip to content

Commit f0c7782

Browse files
authored
chore(19502): change license checker and generator behaviour to only add dev dependencies with --dev flag. (#19626) (#19825)
1 parent 5e4380e commit f0c7782

File tree

3 files changed

+7
-4
lines changed

3 files changed

+7
-4
lines changed

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@
5454
"test:ui:runner": "echo 'use `node scripts/functional_test_runner`' && false",
5555
"test:server": "grunt test:server",
5656
"test:coverage": "grunt test:coverage",
57-
"checkLicenses": "grunt licenses",
57+
"checkLicenses": "grunt licenses --dev",
5858
"build": "node scripts/build",
5959
"start": "sh ./bin/kibana --dev",
6060
"precommit": "node scripts/precommit_hook",

tasks/licenses.js

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,10 +29,13 @@ export default function licenses(grunt) {
2929
const done = this.async();
3030

3131
try {
32+
const dev = Boolean(grunt.option('dev'));
33+
3234
assertLicensesValid({
3335
packages: await getInstalledPackages({
3436
directory: grunt.config.get('root'),
35-
licenseOverrides: LICENSE_OVERRIDES
37+
licenseOverrides: LICENSE_OVERRIDES,
38+
dev
3639
}),
3740
validLicenses: LICENSE_WHITELIST
3841
});

tasks/licenses_csv_report.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -59,12 +59,12 @@ export default function licensesCSVReport(grunt) {
5959
try {
6060
const file = grunt.option('csv');
6161
const directory = grunt.option('directory');
62-
const release = Boolean(grunt.option('release'));
62+
const dev = Boolean(grunt.option('dev'));
6363

6464
const packages = await getInstalledPackages({
6565
directory: directory ? resolve(directory) : grunt.config.get('root'),
6666
licenseOverrides: LICENSE_OVERRIDES,
67-
dev: !release
67+
dev
6868
});
6969

7070
const csv = packages.map(pkg => {

0 commit comments

Comments
 (0)