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

Commit 3ca2516

Browse files
committed
Move config schema to package.json
Move the configuration definition to the package.json so Atom doesn't have to load the package code to know what settings it has available.
1 parent 10b5845 commit 3ca2516

File tree

2 files changed

+37
-36
lines changed

2 files changed

+37
-36
lines changed

lib/main.js

Lines changed: 0 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -4,41 +4,6 @@ let helpers = null;
44
let clangFlags = null;
55

66
export default {
7-
config: {
8-
execPath: {
9-
type: 'string',
10-
default: 'clang',
11-
},
12-
clangIncludePaths: {
13-
type: 'array',
14-
default: ['.'],
15-
},
16-
clangSuppressWarnings: {
17-
type: 'boolean',
18-
default: false,
19-
},
20-
clangDefaultCFlags: {
21-
type: 'string',
22-
default: '-Wall',
23-
},
24-
clangDefaultCppFlags: {
25-
type: 'string',
26-
default: '-Wall -std=c++11',
27-
},
28-
clangDefaultObjCFlags: {
29-
type: 'string',
30-
default: '',
31-
},
32-
clangDefaultObjCppFlags: {
33-
type: 'string',
34-
default: '',
35-
},
36-
clangErrorLimit: {
37-
type: 'integer',
38-
default: 0,
39-
},
40-
},
41-
427
activate() {
438
require('atom-package-deps').install('linter-clang');
449
},

package.json

Lines changed: 37 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,43 @@
99
},
1010
"license": "MIT",
1111
"engines": {
12-
"atom": ">0.50.0"
12+
"atom": ">=1.4.0 <2.0.0"
13+
},
14+
"configSchema": {
15+
"execPath": {
16+
"type": "string",
17+
"default": "clang"
18+
},
19+
"clangIncludePaths": {
20+
"type": "array",
21+
"default": [
22+
"."
23+
]
24+
},
25+
"clangSuppressWarnings": {
26+
"type": "boolean",
27+
"default": false
28+
},
29+
"clangDefaultCFlags": {
30+
"type": "string",
31+
"default": "-Wall"
32+
},
33+
"clangDefaultCppFlags": {
34+
"type": "string",
35+
"default": "-Wall -std=c++11"
36+
},
37+
"clangDefaultObjCFlags": {
38+
"type": "string",
39+
"default": ""
40+
},
41+
"clangDefaultObjCppFlags": {
42+
"type": "string",
43+
"default": ""
44+
},
45+
"clangErrorLimit": {
46+
"type": "integer",
47+
"default": 0
48+
}
1349
},
1450
"scripts": {
1551
"test": "apm test",

0 commit comments

Comments
 (0)