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: remove eslint-config-prettier devDep #506

Merged
merged 2 commits into from
Mar 21, 2023

Conversation

nschonni
Copy link
Contributor

@nschonni nschonni commented Mar 3, 2023

Doesn't appear that ESLint is wired up right now, so the plugin shouldn't be needed

@isaacs
Copy link
Owner

isaacs commented Mar 11, 2023

Just wanted to let you know I haven't forgotten about these, just been busy with actual work coding, and even though these are fairly low-impact, I want to review them carefully, because if they're worthwhile, I'll pull them into the package setup scripts I use as part of npm init isaacs. I'll probably get to that in the next week or two.

@isaacs isaacs force-pushed the remove-eslint-config-prettier branch from e09c252 to 65b9b47 Compare March 21, 2023 00:05
@isaacs isaacs merged commit 65b9b47 into isaacs:main Mar 21, 2023
@isaacs isaacs temporarily deployed to github-pages March 21, 2023 00:05 — with GitHub Actions Inactive
@isaacs
Copy link
Owner

isaacs commented Mar 21, 2023

Thanks!

@nschonni nschonni deleted the remove-eslint-config-prettier branch March 21, 2023 00:27
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

2 participants