-
Notifications
You must be signed in to change notification settings - Fork 192
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
added Inline JS support to ESlint #529
base: master
Are you sure you want to change the base?
Conversation
@harikesh409 @ravikr126 please merge my pr I have done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The automated linting is not done properly do the required changes.
@harikesh409 I will do the changes bu tomorrow will I get scores for this and can u merge as I send the pr before may 31?? |
@harikesh409 @ravikr126 I have done all the changes as told by u and have rectified everything please merge my pr by tomorrow |
Resolve merge conflicts and also a blank line addition is present on readme file remove that also. |
Description
added Inline JS support to ESlint
Fixes #518
Type of change
Checklist: