Skip to content

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

Closed
ghusse wants to merge 1 commit into
brianc:masterfrom
ghusse:chore/upgrade-eslint-fix-ci
Closed

chore: fix the CI by bumping eslint and its dependencies#3110
ghusse wants to merge 1 commit into
brianc:masterfrom
ghusse:chore/upgrade-eslint-fix-ci

Conversation

@ghusse

@ghusse ghusse commented Dec 13, 2023

Copy link
Copy Markdown

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

brianc commented Dec 28, 2023

Copy link
Copy Markdown
Owner

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

ghusse commented Jan 10, 2024

Copy link
Copy Markdown
Author

Thanks!

@abenhamdine

Copy link
Copy Markdown
Contributor

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

@alxndrsn

Copy link
Copy Markdown
Contributor

Closes #3147

@brianc

brianc commented Mar 5, 2024

Copy link
Copy Markdown
Owner

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

brianc commented Mar 6, 2024

Copy link
Copy Markdown
Owner

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.

5 participants