Skip to content

Commit a739f08

Browse files
committed
fix(fypp): erroneous warning for fypp
Previously the extension logged warnings regardless of whether fypp was enabled or not due to the linting compiler being different. Fixes #1017
1 parent 5b1acd0 commit a739f08

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

src/lint/provider.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -135,11 +135,12 @@ export class LinterSettings {
135135

136136
public get fyppEnabled(): boolean {
137137
// FIXME: fypp currently works only with gfortran
138-
if (this.compiler !== 'gfortran') {
138+
const isEnabled = this.config.get<boolean>('linter.fypp.enabled');
139+
if (this.compiler !== 'gfortran' && isEnabled) {
139140
this.logger.warn(`[lint] fypp currently only supports gfortran.`);
140141
return false;
141142
}
142-
return this.config.get<boolean>('linter.fypp.enabled');
143+
return isEnabled;
143144
}
144145
public get fyppPath(): string {
145146
return this.config.get<string>('linter.fypp.path');

0 commit comments

Comments
 (0)