Skip to content

Commit 5f7af33

Browse files
committed
Fix formatting of line comments
1 parent 54cee34 commit 5f7af33

4 files changed

Lines changed: 32 additions & 1 deletion

File tree

server/src/language-service/formatProvider.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ export const jinjaIfEndRegex = /\{%-?\s+endif\s+%}$/;
3333
export const jinjaExpressionRegex = /\{%[\S\s]*%\}/;
3434
export const stopFormattingRegex = /^\{#\s+format:off\s+#}$/;
3535
export const startFormattingRegex = /^\{#\s+format:on\s+#}$/;
36-
export const lineCommentRegex = /^\s*\/\/\s|\*\/\s*$|#}\s*$|%}\s*$/;
36+
export const lineCommentRegex = /\/\/\s|\*\/\s*$|#}\s*$|%}\s*$/;
3737

3838
export class FormattingProvider {
3939
private readonly cnfgProvider: ISepticConfigProvider;
Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
CalcPvr: Test
2+
Text1= ""
3+
Text2= ""
4+
Alg= "1+1" // Test line comment
5+
6+
CalcPvr: Something
7+
Text1= ""
8+
Text2= ""
9+
Alg= "2"
Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
CalcPvr: Test
2+
Text1= ""
3+
Text2= ""
4+
Alg= "1+1" // Test line comment
5+
6+
7+
CalcPvr: Something
8+
Text1= ""
9+
Text2= ""
10+
Alg= "2"

server/src/test/format.test.ts

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -80,4 +80,16 @@ describe("Test formatting", () => {
8080
);
8181
compareFiles(expectedContent, formattedContent);
8282
});
83+
it("Expect correct formatting of line comments with jinja", () => {
84+
const content = loadFile("formatting/lineCommentOriginal.cnfg");
85+
const doc = TextDocument.create("", "", 0, content);
86+
const cnfg = parseSepticSync(doc.getText());
87+
const formatter = new SepticCnfgFormatter(cnfg, doc);
88+
const edits = formatter.format();
89+
const formattedContent = TextDocument.applyEdits(doc, edits);
90+
const expectedContent = loadFile(
91+
"formatting/lineCommentExpected.cnfg"
92+
);
93+
compareFiles(expectedContent, formattedContent);
94+
});
8395
});

0 commit comments

Comments
 (0)