Skip to content

Test PR - DO not comment - feel free to unsubscribe #7

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

Closed
wants to merge 1 commit into from

Conversation

gyandeeps
Copy link
Member

No description provided.

@eslint-deprecated
Copy link

Thanks for the pull request, @gyandeeps! I took a look to make sure it's ready for merging and found some changes are needed:

  • The commit summary needs to begin with a tag (such as Fix: or Update:). Please check out our guide for how to properly format your commit summary and update it on this pull request.

Can you please update the pull request to address these?
(More information can be found in our pull request guide.)

@eslint-deprecated
Copy link

@gyandeeps Looks like you have made some code changes but you have not added any unit tests. We require unit test for all type of code changes.
If this is a mistake then please ignore this message. I am not that smart as a bot (working on it). Thanks 😄

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.

1 participant