Skip to content

fix(deps): update eslint version to fix peer dep conflict #207

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

Open
wants to merge 7 commits into
base: staging
Choose a base branch
from

Conversation

joshuata
Copy link
Contributor

@joshuata joshuata commented Nov 8, 2024

Any change to packages was previously causing issues due to conflicting peer dependencies. This updates to the recommended sveltekit eslint config and recent eslint versions

Any change to packages was previously causing issues due to
conflicting peer dependencies. This updates to the recommended
sveltekit eslint config and recent eslint versions
Copy link

vercel bot commented Nov 8, 2024

@joshuata is attempting to deploy a commit to the satisfactorymodding Team on Vercel.

A member of the Team first needs to authorize it.

joshuata and others added 2 commits November 15, 2024 15:35
Currently the repo requires commitlint to pass, but there is no notification until commit passes.
This adds a prompt when commiting to help devs
Copy link
Member

@budak7273 budak7273 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When trying to run this locally, I see new warnings that haven't been resolved. I'm not sure how your commit passed linting without triggering these warnings - are the new configs not pinned to a specific version of linter rules?

I tried to fix some of these warnings and commit, but it seems like these changes have also made all of my git clients error when trying to skip commit hooks. Not sure if this is a problem on my end or with the changes.

@budak7273 budak7273 moved this from 🆕 New to 👀 In review in Mod Repository (SMR) Mar 30, 2025
@joshuata
Copy link
Contributor Author

I left the warnings since I didn't want to make big changes based on a config change I wasn't sure would be accepted. We can absolutely fix them, I just wanted to be careful

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 👀 In review
Development

Successfully merging this pull request may close these issues.

2 participants