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

Fix lint errors #1107

Merged
merged 3 commits into from
Jun 21, 2024
Merged

Fix lint errors #1107

merged 3 commits into from
Jun 21, 2024

Conversation

raviteja83
Copy link
Contributor

Pull Request Template

Closes #1038

Description

Provide a brief description of the changes made in this pull request.
Eslint config is updated, eslint is updated.

Purpose

Explain the purpose of this pull request and the problem it aims to solve.
This solves the lint errors and husky can run before commiting/pushing.

Solution

Provide a detailed explanation of the solution implemented and any relevant information. Include any code changes or new features.

Screenshots

Include any relevant screenshots or images to showcase the changes made.
image

Checklist

  • Code has been tested and works as intended
  • [ x] Properly documented code
  • Follows coding standards and guidelines
  • All tests pass
  • [] No conflicts with current codebase
  • [] Code has been reviewed by at least one other team member

Related Issues

Link any related issues that this pull request addresses.

Additional Notes

Include any additional notes or information that may be relevant to this pull request.

Contributor Guidelines

Please ensure that your pull request follows the guidelines outlined in the project's CONTRIBUTING.md file.

Thank you for your contribution!

@raviteja83
Copy link
Contributor Author

@alexanmtz I think codacy is showing wrong errors here. could you please take a look. The current usage and formatting is right

@alexanmtz
Copy link
Member

Hey @raviteja83 , thanks for the PR, it looks great 💯

You can ignore the Codacity issues

@alexanmtz alexanmtz merged commit fdd0fbb into worknenjoy:master Jun 21, 2024
1 of 2 checks passed
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.

Fix Lint issues
2 participants