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

@pls-ignore keyword to suppress lint errors #1494

Closed
wants to merge 1 commit into from

Conversation

altintx
Copy link

@altintx altintx commented Aug 10, 2023

If an errant line of code out of linter includes // @pls-ignore it'll be discarded and not reported

closes #1491

@CLAassistant
Copy link

CLAassistant commented Aug 10, 2023

CLA assistant check
All committers have signed the CLA.

@Druue Druue added this to the 5.2.0 milestone Aug 11, 2023
@aqrln aqrln modified the milestones: 5.2.0, 5.3.0 Aug 22, 2023
@altintx
Copy link
Author

altintx commented Sep 29, 2023

Does this need any particular further action from me?

@Druue
Copy link
Contributor

Druue commented Apr 4, 2024

Hey @altintx,

Thanks for submitting this PR! We're currently working on supporting multi-file schema. Do you have a use-case for this functionality that wouldn't already be solved by multi-file schema?

I can't personally imagine any single-file use-cases as even if the VS Code extension isn't complaining, you'll run into errors when using the CLI.

If not, I'll be closing this PR,
Thanks again!

@Druue
Copy link
Contributor

Druue commented Apr 22, 2024

Hey @altintx, again, thank you for submitting this PR, however, with the (understandable) lack of response, we'll be closing this PR now.

@Druue Druue closed this Apr 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Flag to ignore a line in linter
5 participants