Skip to content
This repository has been archived by the owner on Aug 31, 2023. It is now read-only.

benchmark: Add pprettier and dprint to benchmark #3597

Merged
merged 8 commits into from
Nov 9, 2022

Conversation

MichaReiser
Copy link
Contributor

@MichaReiser MichaReiser commented Nov 8, 2022

Summary

Adds the parallel-prettier and an xargs -P setup to the benchmark in response to this tweet

Fixes an issue with the ESLint benchmark.

Test Plan

Rendered Markdown

@MichaReiser MichaReiser requested a review from a team November 8, 2022 17:50
@netlify
Copy link

netlify bot commented Nov 8, 2022

Deploy Preview for docs-rometools canceled.

Name Link
🔨 Latest commit 6ff47cb
🔍 Latest deploy log https://app.netlify.com/sites/docs-rometools/deploys/636bb5b2b836e3000994b8ea

@MichaReiser MichaReiser changed the title benchmark: Add pprettier to comparison benchmark: Add pprettier and dprint to benchmark Nov 9, 2022
benchmark/README.md Outdated Show resolved Hide resolved
@MichaReiser MichaReiser merged commit ec1feb4 into main Nov 9, 2022
@MichaReiser MichaReiser deleted the perf/add-pprettier branch November 9, 2022 15:10
jeysal added a commit to jeysal/rometools that referenced this pull request Nov 9, 2022
* upstream/main:
  fix(rome_js_analyze): Verify method name of React API calls (rome#3619)
  benchmark: Add pprettier and dprint to benchmark (rome#3597)
  feat(vscode): display the version of the language server in the status bar (rome#3616)
  fix(rome_cli): correctly account for diff diagnostics in the printed diagnostics count (rome#3595)
  fix(rome_cli): Respect formatter/linter `enabled` from configuration (rome#3591)
  Remove dead styles
  [website] Add scroll-padding-top
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants