Skip to content

Commit

Permalink
fix: lineEndings should be off by default
Browse files Browse the repository at this point in the history
  • Loading branch information
sabhas committed Feb 20, 2023
1 parent f8b15c7 commit 496e0bc
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
4 changes: 2 additions & 2 deletions sasjslint-schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
"noSpacesInFileNames": true,
"noTabs": true,
"noTrailingSpaces": true,
"lineEndings": "lf",
"lineEndings": "off",
"strictMacroDefinition": true,
"ignoreList": ["sajsbuild", "sasjsresults"]
},
Expand Down Expand Up @@ -185,7 +185,7 @@
"enum": ["lf", "crlf", "off"],
"title": "lineEndings",
"description": "Enforces the configured terminating character for each line. Shows a warning when incorrect line endings are present.",
"default": "lf",
"default": "off",
"examples": ["lf", "crlf"]
},
"strictMacroDefinition": {
Expand Down
2 changes: 2 additions & 0 deletions src/utils/getLintConfig.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ import path from 'path'
import { LintConfig } from '../types/LintConfig'
import { readFile } from '@sasjs/utils/file'
import { getProjectRoot } from './getProjectRoot'
import { LineEndings } from '../types/LineEndings'

export const getDefaultHeader = () =>
`/**{lineEnding} @file{lineEnding} @brief <Your brief here>{lineEnding} <h4> SAS Macros </h4>{lineEnding}**/`
Expand All @@ -10,6 +11,7 @@ export const getDefaultHeader = () =>
* Default configuration that is used when a .sasjslint file is not found
*/
export const DefaultLintConfiguration = {
lineEndings: LineEndings.OFF,
noTrailingSpaces: true,
noEncodedPasswords: true,
hasDoxygenHeader: true,
Expand Down

0 comments on commit 496e0bc

Please sign in to comment.