sasjs/lint
1
0
Fork 0
mirror of https://github.com/sasjs/lint.git synced 2025-12-10 17:34:36 +00:00
Code Issues Projects Releases Wiki Activity

chore: prevent space b/w macroName and starting parentheses is not applicable

Browse Source
This commit is contained in:
Saad Jutt
2021-05-21 20:16:24 +05:00
parent a762dadf37
commit 020a1e08d0
2 changed files with 0 additions and 26 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
src/rules/file/hasMacroParentheses.spec.ts
View File

@@ -139,18 +139,4 @@ describe('hasMacroParentheses', () => {
])
})
})
// it('should return an array with a single diagnostic when a macro definition contains a space', () => {
// const content = `%macro test ()`
// expect(hasMacroParentheses.test(content)).toEqual([
// {
// message: 'Macro definition contains space(s)',
// lineNumber: 1,
// startColumnNumber: 8,
// endColumnNumber: 14,
// severity: Severity.Warning
// }
// ])
// })
})

12
src/rules/file/hasMacroParentheses.ts
View File

@@ -46,18 +46,6 @@ const test = (value: string, config?: LintConfig) => {
1,
severity: Severity.Warning
})
// } else if (macro.name !== macro.name.trim()) {
// diagnostics.push({
// message: 'Macro definition contains space(s)',
// lineNumber: macro.startLineNumber!,
// startColumnNumber: getColumnNumber(macro.declarationLine, macro.name),
// endColumnNumber:
// getColumnNumber(macro.declarationLine, macro.name) +
// macro.name.length -
// 1 +
// `()`.length,
// severity: Severity.Warning
// })
}
})