-
Notifications
You must be signed in to change notification settings - Fork 6.3k
refactor: update prettier and doctoc #5605
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
Conversation
✨ code-server docs for PR #5605 is ready! It will be updated on every commit.
- Host: https://coder.com/docs/code-server/09056b9
- Last deploy status: success
- Commit: d00d3ea
- Workflow status: https://github.com/coder/code-server/actions/runs/3245980952
Codecov Report
Merging #5605 (d00d3ea) into main (d4707d1) will not change coverage.
The diff coverage isn/a.
Additional details and impacted files
@@ Coverage Diff @@ ## main #5605 +/- ## ======================================= Coverage 72.52% 72.52% ======================================= Files 30 30 Lines 1678 1678 Branches 366 366 ======================================= Hits 1217 1217 Misses 398 398 Partials 63 63
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing data
Powered by Codecov. Last update d4707d1...d00d3ea. Read the comment docs.
This does a couple things: - update `.prettierignore` - split `prettier` and `doctoc` commands. you can still run with `yarn fmt` - delete `fmt.sh` and add `doctoc.sh` By doing so, we can run tasks in parallel in CI and we should also have less false positives than before with `yarn fmt` locally.
This modifies the prettier job to use actionsx/prettier. It also adds a job for `doctoc`.
75184c9 to
d8c6c01
Compare
I want to make a couple minor changes so converting to draft
Uh oh!
There was an error while loading. Please reload this page.
This PR refactors our
prettieranddoctoctasks.