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 error #2178

Closed
wants to merge 1 commit into from
Closed

Conversation

sharefe-demo
Copy link

Lint Problem

When you run yarn lint, it will throw error:

yarn lint problem

Copy link

codesandbox-ci bot commented Jun 9, 2024

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Copy link

codesandbox-ci bot commented Jun 9, 2024

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

@timdorr
Copy link
Member

timdorr commented Jun 9, 2024

Oh crap, it looks like ESLint has been busted for a bit and we haven't been running it in CI. Looking into this in some more detail, but it looks like we were relying on a pretest script and support for that was dropped some time ago.

@timdorr
Copy link
Member

timdorr commented Jun 9, 2024

Thank you for looking into this and helping to surface the issue we had in CI. I've got it fixed now on master, but thank you for pointing me in the right direction!

@timdorr timdorr closed this Jun 9, 2024
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.

2 participants