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

Apply fixes from CodeFactor #89

Merged
merged 1 commit into from
Jul 30, 2023
Merged

Apply fixes from CodeFactor #89

merged 1 commit into from
Jul 30, 2023

Conversation

ansulagrawal
Copy link
Member

This pull request applies style fixes from code review by CodeFactor.


For more information, click here.

@DamyanBG
Copy link
Contributor

I can see that is just adding the ESlint rules. We can achieve the same functionality if we add ESlint rules and Prettier formatting. Then we will not need to use 3rd party websites like this one.

P.S. I agree with the changes

Copy link
Contributor

@DamyanBG DamyanBG left a comment

Choose a reason for hiding this comment

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

I agree

@ansulagrawal
Copy link
Member Author

What all configuration do we did default eslint with airbnb one or which one

@ansulagrawal ansulagrawal merged commit d2976d3 into master Jul 30, 2023
@ansulagrawal ansulagrawal deleted the master-cf-autofix branch July 30, 2023 15:56
@DamyanBG
Copy link
Contributor

airbnb with react

@ansulagrawal
Copy link
Member Author

Will you configure it or shall I?

@DamyanBG
Copy link
Contributor

Today is Monday and I am really busy, if you want to be done, do it. If not, I can make it later this week.

@ansulagrawal
Copy link
Member Author

OK, I will start working on it.

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.

3 participants