git.iko.soy/vs-code-ormolu
on master help sign in
Added proper range support
4ca3282iko
CHANGELOG.md+39−4
compare with master
unifiedsplit
@@ -1,9 +1,44 @@
1# Change Log
1# Change Log
2
2
3−All notable changes to the "ormolu" extension will be documented in this file.
4
3
5−Check [Keep a Changelog](http://keepachangelog.com/) for recommendations on how to structure this file.
4+### 0.0.10
6
5
7−## [Unreleased]
6+Fixed off-by-one range bug.
8
7
8+### 0.0.9
9+
10+Fixed range values.
11+
12+### 0.0.8
13+
14+Fixed bug.
15+
16+### 0.0.7
17+
18+Fixed bug.
19+
20+### 0.0.6
21+
22+Added proper range support.
23+
24+### 0.0.5
25+
26+Fixed bugs.
27+
28+### 0.0.4
29+
30+Fixed bugs.
31+
32+### 0.0.3
33+
34+Added logo.
35+
36+### 0.0.2
37+
38+Added configurations.
39+
40+Changed error to warning.
41+
42+### 0.0.1
43+
9−- Initial release
44+Initial release of Ormolu
package.json+155−157
@@ -1,158 +1,156 @@
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.5",
6+ "version": "0.0.10",
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+ "icon": "logo.png",
12− "engines": {
12+ "engines": {
13− "vscode": "^1.39.0"
13+ "vscode": "^1.39.0"
14− },
14+ },
15− "categories": [
15+ "categories": [
16− "Formatters"
16+ "Formatters"
17− ],
17+ ],
18− "main": "./out/extension.js",
18+ "main": "./out/extension.js",
19− "contributes": {
19+ "contributes": {
20− "languages": [
20+ "languages": [{
21− {
22− "id": "haskell",
21+ "id": "haskell",
23− "aliases": [
22+ "aliases": [
24− "Haskell"
23+ "Haskell"
25− ]
24+ ]
26− }
27− ],
25+ }],
28− "configuration": {
26+ "configuration": {
29− "title": "Ormolu",
27+ "title": "Ormolu",
30− "properties": {
28+ "properties": {
31− "ormolu.extensions.AlternativeLayoutRule": {
29+ "ormolu.extensions.AlternativeLayoutRule": {
32− "type": "boolean",
30+ "type": "boolean",
33− "default": false,
31+ "default": false,
34− "description": "Enable the AlternativeLayoutRule extension."
32+ "description": "Enable the AlternativeLayoutRule extension."
35− },
33+ },
36− "ormolu.extensions.AlternativeLayoutRuleTransitional": {
34+ "ormolu.extensions.AlternativeLayoutRuleTransitional": {
37− "type": "boolean",
35+ "type": "boolean",
38− "default": false,
36+ "default": false,
39− "description": "Enable the AlternativeLayoutRuleTransitional extension."
37+ "description": "Enable the AlternativeLayoutRuleTransitional extension."
40− },
38+ },
41− "ormolu.extensions.Arrows": {
39+ "ormolu.extensions.Arrows": {
42− "type": "boolean",
40+ "type": "boolean",
43− "default": false,
41+ "default": false,
44− "description": "Enable the Arrows extension."
42+ "description": "Enable the Arrows extension."
45− },
43+ },
46− "ormolu.extensions.BangPatterns": {
44+ "ormolu.extensions.BangPatterns": {
47− "type": "boolean",
45+ "type": "boolean",
48− "default": false,
46+ "default": false,
49− "description": "Enable the BangPatterns extension."
47+ "description": "Enable the BangPatterns extension."
50− },
48+ },
51− "ormolu.extensions.Cpp": {
49+ "ormolu.extensions.Cpp": {
52− "type": "boolean",
50+ "type": "boolean",
53− "default": false,
51+ "default": false,
54− "description": "Enable the Cpp extension."
52+ "description": "Enable the Cpp extension."
55− },
53+ },
56− "ormolu.extensions.MagicHash": {
54+ "ormolu.extensions.MagicHash": {
57− "type": "boolean",
55+ "type": "boolean",
58− "default": false,
56+ "default": false,
59− "description": "Enable the MagicHash extension."
57+ "description": "Enable the MagicHash extension."
60− },
58+ },
61− "ormolu.extensions.MonadComprehensions": {
59+ "ormolu.extensions.MonadComprehensions": {
62− "type": "boolean",
60+ "type": "boolean",
63− "default": false,
61+ "default": false,
64− "description": "Enable the MonadComprehensions extension."
62+ "description": "Enable the MonadComprehensions extension."
65− },
63+ },
66− "ormolu.extensions.PatternSynonyms": {
64+ "ormolu.extensions.PatternSynonyms": {
67− "type": "boolean",
65+ "type": "boolean",
68− "default": false,
66+ "default": false,
69− "description": "Enable the PatternSynonyms extension."
67+ "description": "Enable the PatternSynonyms extension."
70− },
68+ },
71− "ormolu.extensions.RecursiveDo": {
69+ "ormolu.extensions.RecursiveDo": {
72− "type": "boolean",
70+ "type": "boolean",
73− "default": false,
71+ "default": false,
74− "description": "Enable the RecursiveDo extension."
72+ "description": "Enable the RecursiveDo extension."
75− },
73+ },
76− "ormolu.extensions.StaticPointers": {
74+ "ormolu.extensions.StaticPointers": {
77− "type": "boolean",
75+ "type": "boolean",
78− "default": false,
76+ "default": false,
79− "description": "Enable the StaticPointers extension."
77+ "description": "Enable the StaticPointers extension."
80− },
78+ },
81− "ormolu.extensions.TemplateHaskellQuotes": {
79+ "ormolu.extensions.TemplateHaskellQuotes": {
82− "type": "boolean",
80+ "type": "boolean",
83− "default": false,
81+ "default": false,
84− "description": "Enable the TemplateHaskellQuotes extension."
82+ "description": "Enable the TemplateHaskellQuotes extension."
85− },
83+ },
86− "ormolu.extensions.TransformListComp": {
84+ "ormolu.extensions.TransformListComp": {
87− "type": "boolean",
85+ "type": "boolean",
88− "default": false,
86+ "default": false,
89− "description": "Enable the TransformListComp extension."
87+ "description": "Enable the TransformListComp extension."
90− },
88+ },
91− "ormolu.extensions.TypeApplications": {
89+ "ormolu.extensions.TypeApplications": {
92− "type": "boolean",
90+ "type": "boolean",
93− "default": false,
91+ "default": false,
94− "description": "Enable the TypeApplications extension."
92+ "description": "Enable the TypeApplications extension."
95− },
93+ },
96− "ormolu.extensions.UnboxedSums": {
94+ "ormolu.extensions.UnboxedSums": {
97− "type": "boolean",
95+ "type": "boolean",
98− "default": false,
96+ "default": false,
99− "description": "Enable the UnboxedSums extension."
97+ "description": "Enable the UnboxedSums extension."
100− },
98+ },
101− "ormolu.extensions.UnboxedTuples": {
99+ "ormolu.extensions.UnboxedTuples": {
102− "type": "boolean",
100+ "type": "boolean",
103− "default": false,
101+ "default": false,
104− "description": "Enable the UnboxedTuples extension."
102+ "description": "Enable the UnboxedTuples extension."
105− },
103+ },
106− "ormolu.extensions.UnicodeSyntax": {
104+ "ormolu.extensions.UnicodeSyntax": {
107− "type": "boolean",
105+ "type": "boolean",
108− "default": false,
106+ "default": false,
109− "description": "Enable the UnicodeSyntax extension."
107+ "description": "Enable the UnicodeSyntax extension."
110− },
108+ },
111− "ormolu.tolerateCpp": {
109+ "ormolu.tolerateCpp": {
112− "type": "boolean",
110+ "type": "boolean",
113− "default": false,
111+ "default": false,
114− "description": "Do not fail if CPP pragma is present."
112+ "description": "Do not fail if CPP pragma is present."
115− },
113+ },
116− "ormolu.unsafe": {
114+ "ormolu.unsafe": {
117− "type": "boolean",
115+ "type": "boolean",
118− "default": false,
116+ "default": false,
119− "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."
120− },
118+ },
121− "ormolu.checkIdempotency": {
119+ "ormolu.checkIdempotency": {
122− "type": "boolean",
120+ "type": "boolean",
123− "default": false,
121+ "default": false,
124− "description": "Fail if formatting is not idempotent."
122+ "description": "Fail if formatting is not idempotent."
125− },
123+ },
126− "ormolu.customArguments": {
124+ "ormolu.customArguments": {
127− "type": [
125+ "type": [
128− "string",
126+ "string",
129− "null"
127+ "null"
130− ],
128+ ],
131− "default": null,
129+ "default": null,
132− "description": "Custom arguments to pass to ormolu."
130+ "description": "Custom arguments to pass to ormolu."
133− }
131+ }
134− }
132+ }
135− }
133+ }
136− },
134+ },
137− "activationEvents": [
135+ "activationEvents": [
138− "onLanguage:haskell"
136+ "onLanguage:haskell"
139− ],
137+ ],
140− "scripts": {
138+ "scripts": {
141− "vscode:prepublish": "yarn run compile",
139+ "vscode:prepublish": "yarn run compile",
142− "compile": "tsc -p ./",
140+ "compile": "tsc -p ./",
143− "watch": "tsc -watch -p ./",
141+ "watch": "tsc -watch -p ./",
144− "pretest": "yarn run compile",
142+ "pretest": "yarn run compile",
145− "test": "node ./out/test/runTest.js"
143+ "test": "node ./out/test/runTest.js"
146− },
144+ },
147− "devDependencies": {
145+ "devDependencies": {
148− "@types/glob": "^7.1.1",
146+ "@types/glob": "^7.1.1",
149− "@types/mocha": "^5.2.6",
147+ "@types/mocha": "^5.2.6",
150− "@types/node": "^10.12.21",
148+ "@types/node": "^10.12.21",
151− "@types/vscode": "^1.39.0",
149+ "@types/vscode": "^1.39.0",
152− "glob": "^7.1.4",
150+ "glob": "^7.1.4",
153− "mocha": "^6.1.4",
151+ "mocha": "^6.1.4",
154− "typescript": "^3.3.1",
152+ "typescript": "^3.3.1",
155− "tslint": "^5.12.1",
153+ "tslint": "^5.12.1",
156− "vscode-test": "^1.2.0"
154+ "vscode-test": "^1.2.0"
157− }
155+ }
158−}
156+}
src/extension.ts+13−4
@@ -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
@@ -25,13 +25,15 @@ export function activate(context: vscode.ExtensionContext) {
25
25
26 let command = "ormolu";
26 let command = "ormolu";
27
27
28+ command += ` --start-line ${range.start.line + 1} --end-line ${range.end.line + (range.end.character == 0 ? 0 : 1)}`
29+
28 for (let [key, value] of Object.entries(cfg.extensions)) {
30 for (let [key, value] of Object.entries(cfg.extensions)) {
29 if (value) {
31 if (value) {
30 command += " -o -X" + key;
32 command += " -o -X" + key;
31 }
33 }
32 }
34 }
33
35
34− if (!isNull(cfg.customArguments)) {
36+ if (cfg.customArguments !== null) {
35 command += " " + cfg.customArguments;
37 command += " " + cfg.customArguments;
36 }
38 }
37
39
@@ -47,14 +49,21 @@ export function activate(context: vscode.ExtensionContext) {
47 command += " -c";
49 command += " -c";
48 }
50 }
49
51
50− const text = document.getText(range);
52+ const text = document.getText();
51 let formattedText;
53 let formattedText;
52 try {
54 try {
53 formattedText = proc.execSync(command, { input: text }).toString();
55 formattedText = proc.execSync(command, { input: text }).toString();
54 } catch (e) {
56 } catch (e) {
55 return showErrorMessage('Ormolu failed to format the code', e);
57 return showErrorMessage('Ormolu failed to format the code', e);
56 }
58 }
59+
60+ const fullDocumentRange =
61+ new Range
62+ (document.lineAt(0).range.start
63+ , document.lineAt(document.lineCount - 1).range.end
64+ );
65+
57− return [vscode.TextEdit.replace(range, formattedText)];
66+ return [vscode.TextEdit.replace(fullDocumentRange, formattedText)];
58 }
67 }
59 });
68 });
60}
69}