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

Commit 01f260e

Browse files
committed
Install dependencies on activation.
@steelbrain
1 parent f9c4175 commit 01f260e

2 files changed

Lines changed: 8 additions & 19 deletions

File tree

lib/main.js

Lines changed: 3 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -16,24 +16,9 @@ export default {
1616
},
1717

1818
activate: () => {
19-
// Because all of the grammars this linter supports are
20-
// built into the editor we do not need to throw errors when
21-
// any one of the grammmars isn't installed. If a user has the grammar
22-
// disabled that is a choice they have made.
23-
24-
// Show the user an error if they do not have an appropriate linter base
25-
// package installed from Atom Package Manager. This will not be an issues
26-
// after a base linter package is integrated into Atom, in the comming
27-
// months.
28-
// TODO: Remove when Linter Base is integrated into Atom.
29-
if (!atom.packages.getLoadedPackages("linter")) {
30-
atom.notifications.addError(
31-
"Linter package not found.",
32-
{
33-
detail: "Please install the `linter` package in your Settings view."
34-
}
35-
);
36-
}
19+
// We are now using steelbrain's package dependency package to install our
20+
// dependencies.
21+
require("atom-package-deps").install("linter-ruby");
3722
},
3823

3924
provideLinter: () => {

package.json

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,11 +16,15 @@
1616
}
1717
},
1818
"dependencies": {
19-
"atom-linter": "^3.0.0"
19+
"atom-linter": "^3.0.0",
20+
"atom-package-deps": "^2.0.0"
2021
},
2122
"devDependencies": {
2223
"eslint": "latest"
2324
},
25+
"package-deps": [
26+
"linter"
27+
],
2428
"eslintConfig": {
2529
"env": {
2630
"es6": true,

0 commit comments

Comments
 (0)