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

chore: Move Prettier compat metrics to the top #2765

Merged
merged 1 commit into from
Jun 23, 2022

Conversation

IWANABETHATGUY
Copy link
Contributor

Summary

  1. Moving the overall metrics back to the top.

Test Plan

  1. No need to test, I guess?

@IWANABETHATGUY
Copy link
Contributor Author

@MichaReiser, would you mind helping with reviewing?

@MichaReiser
Copy link
Contributor

Looks good. Would you mind sharing the header of the new report.md file (like the first 20 lines or so)

@IWANABETHATGUY
Copy link
Contributor Author

**File Based Average Prettier Similarity**: 75.24%  
**Line Based Average Prettier Similarity**: 70.33%  
the definition of similarity you could found here: https://github.com/rome/tools/issues/2555#issuecomment-1124787893
# js/array-spread/multiple.js
**Prettier Similarity**: 100.00%
# js/arrays/empty.js
```diff
-const a =
-  someVeryVeryVeryVeryVeryVeryVeryVeryVeryVeryVeryVeLong.Expression || [];
-const b =
-  someVeryVeryVeryVeryVeryVeryVeryVeryVeryVeryVeryVeLong.Expression || {};
+const a = someVeryVeryVeryVeryVeryVeryVeryVeryVeryVeryVeryVeLong.Expression || [];
+const b = someVeryVeryVeryVeryVeryVeryVeryVeryVeryVeryVeryVeLong.Expression || {};
```
**Prettier Similarity**: 0.00%
# js/arrays/holes-in-args.js
```diff
-new Test().test().test([, 0]).test();
+new Test()
+  .test()
+  .test([, 0])
+  .test();
```
**Prettier Similarity**: 0.00%

@MichaReiser MichaReiser changed the title fix: adjust report_md content chore: Move Prettier compat metrics to the top Jun 23, 2022
@MichaReiser MichaReiser merged commit 3235d35 into rome:main Jun 23, 2022
@IWANABETHATGUY IWANABETHATGUY deleted the fix/pretter_test_order branch June 23, 2022 15:47
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.

None yet

2 participants