Skip to content

ESLint warning fixes#87

Merged
nehabagdia merged 5 commits into
mainfrom
fix/eslint-warning-fixes
Mar 13, 2024
Merged

ESLint warning fixes#87
nehabagdia merged 5 commits into
mainfrom
fix/eslint-warning-fixes

Conversation

@Deepak-Kesavan

Copy link
Copy Markdown
Contributor

What

Fixed eslint warnings

Why

How

Database Migrations

Env Config

Relevant Docs

Related Issues or PRs

Dependencies Versions

Notes on Testing

  • Ran npm run build and the logs didn't show any eslint warnings

Screenshots

image

Checklist

I have read and understood the Contribution Guidelines.

@hari-kuriakose hari-kuriakose left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@Deepak-Kesavan LGTM overall.

@tahierhussain tahierhussain left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM. Please resolve the conflicts.

@nehabagdia
nehabagdia merged commit fa5ee22 into main Mar 13, 2024
@nehabagdia
nehabagdia deleted the fix/eslint-warning-fixes branch March 13, 2024 06:19
pk-zipstack pushed a commit that referenced this pull request Aug 20, 2025
* Fixed eslint warnings

* Fixed eslint warnings after merging the latest code

* Removed unused import
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.

4 participants