on master help sign in
CHANGELOG.md+46−4
since splitsnapshots
unifiedsplit
@@ -1,9 +1,51 @@
1# Change Log
1# Change Log
2
2
3−All notable changes to the "ormolu" extension will be documented in this file.
3+### 0.0.12
4
4
5−Check [Keep a Changelog](http://keepachangelog.com/) for recommendations on how to structure this file.
5+Fixed a bug where executable paths outside of `$PATH` would fail. (@lzszt)
6
6
7−## [Unreleased]
7+### 0.0.11
8
8
9+Added the ability to specify the path to the executable.
10+
11+### 0.0.10
12+
13+Fixed off-by-one range bug.
14+
15+### 0.0.9
16+
17+Fixed range values.
18+
19+### 0.0.8
20+
21+Fixed bug.
22+
23+### 0.0.7
24+
25+Fixed bug.
26+
27+### 0.0.6
28+
29+Added proper range support.
30+
31+### 0.0.5
32+
33+Fixed bugs.
34+
35+### 0.0.4
36+
37+Fixed bugs.
38+
39+### 0.0.3
40+
41+Added logo.
42+
43+### 0.0.2
44+
45+Added configurations.
46+
47+Changed error to warning.
48+
49+### 0.0.1
50+
9−- Initial release
51+Initial release of Ormolu
README.md+12
@@ -21,3 +21,15 @@ Initial release of Ormolu
21Added configurations.
21Added configurations.
22
22
23Changed error to warning.
23Changed error to warning.
24+
25+### 0.0.3
26+
27+Added logo.
28+
29+### 0.0.4
30+
31+Fixed bugs.
32+
33+### 0.0.5
34+
35+Fixed bugs.
logo.pngnew file

Binary file not shown.

package.json+161−155
@@ -1,157 +1,163 @@
1{
1{
2− "name": "ormolu",
2+ "name": "ormolu",
3− "displayName": "Ormolu",
3+ "displayName": "Ormolu",
4− "publisher": "ilyakooo0",
4+ "publisher": "ilyakooo0",
5− "description": "Haskell code formatter with ormolu",
5+ "description": "Haskell code formatter with ormolu",
6− "version": "0.0.2",
6+ "version": "0.0.12",
7− "repository": {
7+ "repository": {
8− "type": "git",
8+ "type": "git",
9− "url": "https://github.com/ilyakooo0/vs-code-ormolu"
9+ "url": "https://github.com/ilyakooo0/vs-code-ormolu"
10− },
10+ },
11+ "icon": "logo.png",
11− "engines": {
12+ "engines": {
12− "vscode": "^1.39.0"
13+ "vscode": "^1.39.0"
13− },
14+ },
14− "categories": [
15+ "categories": [
15− "Formatters"
16+ "Formatters"
16− ],
17+ ],
17− "main": "./out/extension.js",
18+ "main": "./out/extension.js",
18− "contributes": {
19+ "contributes": {
19− "languages": [
20+ "languages": [{
20− {
21− "id": "haskell",
21+ "id": "haskell",
22− "aliases": [
22+ "aliases": [
23− "Haskell"
23+ "Haskell"
24− ]
24+ ]
25− }
26− ],
25+ }],
27− "configuration": {
26+ "configuration": {
28− "title": "Ormolu",
27+ "title": "Ormolu",
29− "properties": {
28+ "properties": {
30− "ormolu.extensions.AlternativeLayoutRule": {
29+ "ormolu.extensions.AlternativeLayoutRule": {
31− "type": "boolean",
30+ "type": "boolean",
32− "default": false,
31+ "default": false,
33− "description": "Enable the AlternativeLayoutRule extension."
32+ "description": "Enable the AlternativeLayoutRule extension."
34− },
33+ },
35− "ormolu.extensions.AlternativeLayoutRuleTransitional": {
34+ "ormolu.extensions.AlternativeLayoutRuleTransitional": {
36− "type": "boolean",
35+ "type": "boolean",
37− "default": false,
36+ "default": false,
38− "description": "Enable the AlternativeLayoutRuleTransitional extension."
37+ "description": "Enable the AlternativeLayoutRuleTransitional extension."
39− },
38+ },
40− "ormolu.extensions.Arrows": {
39+ "ormolu.extensions.Arrows": {
41− "type": "boolean",
40+ "type": "boolean",
42− "default": false,
41+ "default": false,
43− "description": "Enable the Arrows extension."
42+ "description": "Enable the Arrows extension."
44− },
43+ },
45− "ormolu.extensions.BangPatterns": {
44+ "ormolu.extensions.BangPatterns": {
46− "type": "boolean",
45+ "type": "boolean",
47− "default": false,
46+ "default": false,
48− "description": "Enable the BangPatterns extension."
47+ "description": "Enable the BangPatterns extension."
49− },
48+ },
50− "ormolu.extensions.Cpp": {
49+ "ormolu.extensions.Cpp": {
51− "type": "boolean",
50+ "type": "boolean",
52− "default": false,
51+ "default": false,
53− "description": "Enable the Cpp extension."
52+ "description": "Enable the Cpp extension."
54− },
53+ },
55− "ormolu.extensions.MagicHash": {
54+ "ormolu.extensions.MagicHash": {
56− "type": "boolean",
55+ "type": "boolean",
57− "default": false,
56+ "default": false,
58− "description": "Enable the MagicHash extension."
57+ "description": "Enable the MagicHash extension."
59− },
58+ },
60− "ormolu.extensions.MonadComprehensions": {
59+ "ormolu.extensions.MonadComprehensions": {
61− "type": "boolean",
60+ "type": "boolean",
62− "default": false,
61+ "default": false,
63− "description": "Enable the MonadComprehensions extension."
62+ "description": "Enable the MonadComprehensions extension."
64− },
63+ },
65− "ormolu.extensions.PatternSynonyms": {
64+ "ormolu.extensions.PatternSynonyms": {
66− "type": "boolean",
65+ "type": "boolean",
67− "default": false,
66+ "default": false,
68− "description": "Enable the PatternSynonyms extension."
67+ "description": "Enable the PatternSynonyms extension."
69− },
68+ },
70− "ormolu.extensions.RecursiveDo": {
69+ "ormolu.extensions.RecursiveDo": {
71− "type": "boolean",
70+ "type": "boolean",
72− "default": false,
71+ "default": false,
73− "description": "Enable the RecursiveDo extension."
72+ "description": "Enable the RecursiveDo extension."
74− },
73+ },
75− "ormolu.extensions.StaticPointers": {
74+ "ormolu.extensions.StaticPointers": {
76− "type": "boolean",
75+ "type": "boolean",
77− "default": false,
76+ "default": false,
78− "description": "Enable the StaticPointers extension."
77+ "description": "Enable the StaticPointers extension."
79− },
78+ },
80− "ormolu.extensions.TemplateHaskellQuotes": {
79+ "ormolu.extensions.TemplateHaskellQuotes": {
81− "type": "boolean",
80+ "type": "boolean",
82− "default": false,
81+ "default": false,
83− "description": "Enable the TemplateHaskellQuotes extension."
82+ "description": "Enable the TemplateHaskellQuotes extension."
84− },
83+ },
85− "ormolu.extensions.TransformListComp": {
84+ "ormolu.extensions.TransformListComp": {
86− "type": "boolean",
85+ "type": "boolean",
87− "default": false,
86+ "default": false,
88− "description": "Enable the TransformListComp extension."
87+ "description": "Enable the TransformListComp extension."
89− },
88+ },
90− "ormolu.extensions.TypeApplications": {
89+ "ormolu.extensions.TypeApplications": {
91− "type": "boolean",
90+ "type": "boolean",
92− "default": false,
91+ "default": false,
93− "description": "Enable the TypeApplications extension."
92+ "description": "Enable the TypeApplications extension."
94− },
93+ },
95− "ormolu.extensions.UnboxedSums": {
94+ "ormolu.extensions.UnboxedSums": {
96− "type": "boolean",
95+ "type": "boolean",
97− "default": false,
96+ "default": false,
98− "description": "Enable the UnboxedSums extension."
97+ "description": "Enable the UnboxedSums extension."
99− },
98+ },
100− "ormolu.extensions.UnboxedTuples": {
99+ "ormolu.extensions.UnboxedTuples": {
101− "type": "boolean",
100+ "type": "boolean",
102− "default": false,
101+ "default": false,
103− "description": "Enable the UnboxedTuples extension."
102+ "description": "Enable the UnboxedTuples extension."
104− },
103+ },
105− "ormolu.extensions.UnicodeSyntax": {
104+ "ormolu.extensions.UnicodeSyntax": {
106− "type": "boolean",
105+ "type": "boolean",
107− "default": false,
106+ "default": false,
108− "description": "Enable the UnicodeSyntax extension."
107+ "description": "Enable the UnicodeSyntax extension."
109− },
108+ },
110− "ormolu.tolerateCpp": {
109+ "ormolu.tolerateCpp": {
111− "type": "boolean",
110+ "type": "boolean",
112− "default": false,
111+ "default": false,
113− "description": "Do not fail if CPP pragma is present."
112+ "description": "Do not fail if CPP pragma is present."
114− },
113+ },
115− "ormolu.unsafe": {
114+ "ormolu.unsafe": {
116− "type": "boolean",
115+ "type": "boolean",
117− "default": false,
116+ "default": false,
118− "description": "Run in \"unsafe\" mode. Do formatting faster but without automatic detection of defects."
117+ "description": "Run in \"unsafe\" mode. Do formatting faster but without automatic detection of defects."
119− },
118+ },
120− "ormolu.checkIdempotency": {
119+ "ormolu.checkIdempotency": {
121− "type": "boolean",
120+ "type": "boolean",
122− "default": false,
121+ "default": false,
123− "description": "Fail if formatting is not idempotent."
122+ "description": "Fail if formatting is not idempotent."
124− },
123+ },
125− "ormolu.customArguments": {
124+ "ormolu.customArguments": {
126− "type": [
125+ "type": [
127− "string",
126+ "string",
128− "null"
127+ "null"
129− ],
128+ ],
130− "default": null,
129+ "default": null,
131− "description": "Custom arguments to pass to ormolu."
130+ "description": "Custom arguments to pass to ormolu."
132− }
131+ },
132+ "ormolu.executablePath": {
133+ "type": [
134+ "string"
135+ ],
136+ "default": "ormolu",
137+ "description": "The path to the ormolu executable to use."
133− }
138+ }
134− }
139+ }
135− },
140+ }
141+ },
136− "activationEvents": [
142+ "activationEvents": [
137− "onLanguage:haskell"
143+ "onLanguage:haskell"
138− ],
144+ ],
139− "scripts": {
145+ "scripts": {
140− "vscode:prepublish": "yarn run compile",
146+ "vscode:prepublish": "yarn run compile",
141− "compile": "tsc -p ./",
147+ "compile": "tsc -p ./",
142− "watch": "tsc -watch -p ./",
148+ "watch": "tsc -watch -p ./",
143− "pretest": "yarn run compile",
149+ "pretest": "yarn run compile",
144− "test": "node ./out/test/runTest.js"
150+ "test": "node ./out/test/runTest.js"
145− },
151+ },
146− "devDependencies": {
152+ "devDependencies": {
147− "@types/glob": "^7.1.1",
153+ "@types/glob": "^7.1.1",
148− "@types/mocha": "^5.2.6",
154+ "@types/mocha": "^5.2.6",
149− "@types/node": "^10.12.21",
155+ "@types/node": "^10.12.21",
150− "@types/vscode": "^1.39.0",
156+ "@types/vscode": "^1.39.0",
151− "glob": "^7.1.4",
157+ "glob": "^7.1.4",
152− "mocha": "^6.1.4",
158+ "mocha": "^6.1.4",
153− "typescript": "^3.3.1",
159+ "typescript": "^3.3.1",
154− "tslint": "^5.12.1",
160+ "tslint": "^5.12.1",
155− "vscode-test": "^1.2.0"
161+ "vscode-test": "^1.2.0"
156− }
162+ }
157}
163}
src/extension.ts+18−8
@@ -2,7 +2,7 @@
2// Import the module and reference it with the alias vscode in your code below
2// Import the module and reference it with the alias vscode in your code below
3import * as vscode from 'vscode';
3import * as vscode from 'vscode';
4import * as proc from 'child_process';
4import * as proc from 'child_process';
5−import { isNull } from 'util';
5+import { Range } from 'vscode';
6
6
7// this method is called when your extension is activated
7// this method is called when your extension is activated
8// your extension is activated the very first time the command is executed
8// your extension is activated the very first time the command is executed
@@ -15,23 +15,26 @@ export function activate(context: vscode.ExtensionContext) {
15 return [];
15 return [];
16 };
16 };
17
17
18+ const cfg = vscode.workspace.getConfiguration("ormolu");
19+
18 try {
20 try {
19− proc.execSync('ormolu --help');
21+ proc.execSync(cfg.executablePath + ' --help');
20 } catch (e) {
22 } catch (e) {
21 return showErrorMessage("Ormolu is not installed", e);
23 return showErrorMessage("Ormolu is not installed", e);
22 }
24 }
23
25
24− const cfg = vscode.workspace.getConfiguration("ormolu");
26+ let command = cfg.executablePath;
25
27
26− let command = "ormolu";
28+ command += ` --start-line ${range.start.line + 1} --end-line ${range.end.line + (range.end.character == 0 ? 0 : 1)}`
29+ command += ` --stdin-input-file ${document.fileName}`
27
30
28 for (let [key, value] of Object.entries(cfg.extensions)) {
31 for (let [key, value] of Object.entries(cfg.extensions)) {
29 if (value) {
32 if (value) {
30− command += " -X" + key;
33+ command += " -o -X" + key;
31 }
34 }
32 }
35 }
33
36
34− if (!isNull(cfg.customArguments)) {
37+ if (cfg.customArguments !== null) {
35 command += " " + cfg.customArguments;
38 command += " " + cfg.customArguments;
36 }
39 }
37
40
@@ -47,14 +50,21 @@ export function activate(context: vscode.ExtensionContext) {
47 command += " -c";
50 command += " -c";
48 }
51 }
49
52
50− const text = document.getText(range);
53+ const text = document.getText();
51 let formattedText;
54 let formattedText;
52 try {
55 try {
53 formattedText = proc.execSync(command, { input: text }).toString();
56 formattedText = proc.execSync(command, { input: text }).toString();
54 } catch (e) {
57 } catch (e) {
55 return showErrorMessage('Ormolu failed to format the code', e);
58 return showErrorMessage('Ormolu failed to format the code', e);
56 }
59 }
60+
61+ const fullDocumentRange =
62+ new Range
63+ (document.lineAt(0).range.start
64+ , document.lineAt(document.lineCount - 1).range.end
65+ );
66+
57− return [vscode.TextEdit.replace(range, formattedText)];
67+ return [vscode.TextEdit.replace(fullDocumentRange, formattedText)];
58 }
68 }
59 });
69 });
60}
70}