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

Add eslint-disable comment not to affect eslint format #36

Merged
merged 1 commit into from Feb 8, 2024

Conversation

getty104
Copy link
Member

@getty104 getty104 commented Feb 1, 2024

What

Now, generated files are affected by eslint formatter.
But, the files should not be formatted by default.
This PR is for avoiding eslint formatting.

@getty104 getty104 force-pushed the add-eslint-disable-not-to-affect-eslint-format branch from b882fc5 to fc0f583 Compare February 1, 2024 08:03
Copy link

codeclimate bot commented Feb 1, 2024

Code Climate has analyzed commit fc0f583 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (80% is the threshold).

This pull request will bring the total coverage in the repository to 99.6% (0.0% change).

View more on Code Climate.

@getty104 getty104 self-assigned this Feb 1, 2024
Copy link

@acronhub acronhub left a comment

Choose a reason for hiding this comment

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

📝
https://eslint.org/docs/latest/use/configure/rules#disabling-rules

To disable rule warnings in an entire file, put a /* eslint-disable */ block comment at the top of the file:

LGTM

@getty104 getty104 merged commit b8840ff into master Feb 8, 2024
8 of 13 checks passed
@getty104 getty104 deleted the add-eslint-disable-not-to-affect-eslint-format branch February 8, 2024 06:54
@getty104 getty104 mentioned this pull request Feb 8, 2024
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.

None yet

2 participants