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

chore: fix the CI by bumping eslint and its dependencies #3110

Closed
wants to merge 1 commit into from

Conversation

ghusse
Copy link

@ghusse ghusse commented Dec 13, 2023

The current linting script is broken in the CI because of the recent bump of prettier that does not seem compatible with the installed version of eslint.

This PR upgrades eslint and its plugins as well, and fixes introduced linting issues on 2 files.

It fixes the current linting script and allows the CI to pass

@brianc
Copy link
Owner

brianc commented Dec 28, 2023

ironically the CI run failed here 🙃 Regardless...I appreciate your effort & help here. I'll get this running. Maybe there's a race condition in the test or something? Re-running now. ❤️

@ghusse
Copy link
Author

ghusse commented Jan 10, 2024

Thanks!

@abenhamdine
Copy link
Contributor

@brianc we need to merge this, because currenty every PR is failing in CI because of the said issue

@alxndrsn
Copy link
Contributor

Closes #3147

@brianc
Copy link
Owner

brianc commented Mar 5, 2024

thanks! I merged another fix for this today - sorry for missing this! I will be more on top of stuff going forward for sure. Been dealing w/ a lot of work stuff as I spin my current role down.

@brianc
Copy link
Owner

brianc commented Mar 6, 2024

I think this might be okay to close now (though the effort is much appreciate) as CI is now green?

Its unfortunate how some version bumps caused such a catastrophe. Espehs while I was in the midst of a bit of a life change situation. Will stay more on top of that in the future 🔨

@ghusse ghusse closed this Mar 6, 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.

None yet

5 participants