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

Replace pretty quick with lint staged #17

Merged
merged 2 commits into from
Dec 15, 2023

Conversation

Roozenboom
Copy link
Owner

Current Behavior

Since pretty-quick is not working with prettier 3 (see issue) and isn't updated for 2 years, it better to replace it with lint-staged.

Expected Behavior

  • replace pretty-quick with lint-staged
  • update prettier to version 3
  • update files format with prettier 3

Related Issue(s)

Fixes #

Copy link

nx-cloud bot commented Dec 15, 2023

☁️ Nx Cloud Report

CI is running/has finished running commands for commit a889c9b. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


✅ Successfully ran 2 targets

Sent with 💌 from NxCloud.

@Roozenboom Roozenboom merged commit 66fd433 into main Dec 15, 2023
8 checks passed
@Roozenboom Roozenboom deleted the replace-pretty-quick-with-lint-staged branch December 15, 2023 10:37
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.

1 participant