Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

implement exceptions using pragmas in comments. #11

Open
allanbowe opened this issue Mar 26, 2021 · 1 comment
Open

implement exceptions using pragmas in comments. #11

allanbowe opened this issue Mar 26, 2021 · 1 comment

Comments

@allanbowe
Copy link
Member

allanbowe commented Mar 26, 2021

There are cases when you have a valid exception to a linter rule, eg the inclusion of a URL over the 80 char line width.

We need a way to apply exceptions to rules beyond simply switching that rule off.

We can achieve that by adding a "pragma" - a marker inside a comment block, eg:

/*
@sasjs_lint_exception <ruleName> <scope>
*/

This has the double benefit that we can create a doxygen marker so that all the linting exceptions are listed alongside the regular HTML documentation (sasjs doc).

@allanbowe allanbowe changed the title Enable linter exceptions Refactor .sasjslint structure and enable setting-specific exceptions / severity level Mar 26, 2021
@krishna-acondy
Copy link
Contributor

krishna-acondy commented Mar 29, 2021

As discussed with @allanbowe , we will implement exceptions using pragmas in comments.

/*
@sasjs_lint_exception <ruleName> <scope>
*/

@allanbowe allanbowe changed the title Refactor .sasjslint structure and enable setting-specific exceptions / severity level implement exceptions using pragmas in comments. Mar 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants