Skip to content

Conversation

@woshiarui
Copy link
Contributor

Add a new configuration item to the plugin:ignoreSyntaxErrorMsg
it is used to ignore specific syntax error messages during detection. You can pass in one or more syntax error messages to ignore.

README.md Outdated
});
```
### ignoreSyntaxErrorMsg
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about renaming this option to excludeErrorLogs?

  • Align with excludeOutput option
  • Shorter and more intuitive

Copy link
Contributor Author

@woshiarui woshiarui Dec 6, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah, it is better. I have already revised it, please take a look at it when you have time~thanks

@woshiarui woshiarui changed the title chore: ignore syntax error msg chore: excludeErrorLogs Dec 6, 2024
@chenjiahan chenjiahan changed the title chore: excludeErrorLogs feat: add excludeErrorLogs option Dec 6, 2024
@chenjiahan
Copy link
Member

Thank you!

@chenjiahan chenjiahan merged commit 45ac950 into rstackjs:main Dec 6, 2024
2 checks passed
@chenjiahan
Copy link
Member

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

Successfully merging this pull request may close these issues.

3 participants