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 checks failing on develop #10344
Conversation
Hi, @nishantwrp, this pull request does not have a "CHANGELOG: ..." label as mentioned in the PR checkbox list. Please add this label. PRs without this label will not be merged. If you are unsure of which label to add, please ask the reviewers for guidance. Thanks! |
Assigning @vinitamurthi, @aks681 for the first pass review of this PR. Thanks! |
Hi @seanlip, can we merge this asap because the lint checks are failing on develop |
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.
Thanks, LGTM.
@nishantwrp how did this breakage happen?
Also @nishantwrp, next time, please could you provide the following info for this sort of PR?
That will save some back-and-forth to find out the details. Thanks! |
Actually, my pr that introduced the eslint checks passed the ci checks and it was being reviewed by the code owners but meanwhile, contributors added more instances of |
Ah ok, makes sense. Can you link to the PR? (In general, try to always provide links, refs, etc. for an audience who is not familiar with what you're talking about.) |
PR #10187 |
Overview
Essential Checklist
PR Pointers