-
-
Notifications
You must be signed in to change notification settings - Fork 4.6k
Comparing changes
Open a pull request
base repository: prettier/prettier
base: 0.20.0
head repository: prettier/prettier
compare: 0.21.0
- 19 commits
- 38 files changed
- 3 contributors
Commits on Feb 28, 2017
-
Configuration menu - View commit details
-
Copy full SHA for aef1ac6 - Browse repository at this point
Copy the full SHA aef1ac6View commit details
Commits on Mar 1, 2017
-
Configuration menu - View commit details
-
Copy full SHA for f1548e7 - Browse repository at this point
Copy the full SHA f1548e7View commit details -
Configuration menu - View commit details
-
Copy full SHA for 198a9e4 - Browse repository at this point
Copy the full SHA 198a9e4View commit details -
[JSX] Don't wrap JSX Elements in parentheses if they are inside JSX E…
…xpression Containers (#845)
Configuration menu - View commit details
-
Copy full SHA for 475208d - Browse repository at this point
Copy the full SHA 475208dView commit details
Commits on Mar 3, 2017
-
Fix comment after the last argument of a function (#856)
The issue is that the comment algorithm attaches it to the return type / value but we want to attach it to the previous one. Fixes #612
Configuration menu - View commit details
-
Copy full SHA for d90af42 - Browse repository at this point
Copy the full SHA d90af42View commit details -
Configuration menu - View commit details
-
Copy full SHA for a91bb5f - Browse repository at this point
Copy the full SHA a91bb5fView commit details -
Configuration menu - View commit details
-
Copy full SHA for 705ac7d - Browse repository at this point
Copy the full SHA 705ac7dView commit details -
Stabilize import as comments (#855)
Since this is extremely rare, I just took the easy way out and if you are adding a comment inside the "as" node, I just move it outside. We could be more fancy but that works. Fixes #620
Configuration menu - View commit details
-
Copy full SHA for 516f590 - Browse repository at this point
Copy the full SHA 516f590View commit details -
Configuration menu - View commit details
-
Copy full SHA for 4b237d3 - Browse repository at this point
Copy the full SHA 4b237d3View commit details -
Inline last arg function arguments (#847)
```js SuperSuperSuperSuperSuperSuperSuperSuperSuperSuperSuperSuperLongCall((err, result) => { // comment }); ``` currently breaks into ```js SuperSuperSuperSuperSuperSuperSuperSuperSuperSuperSuperSuperLongCall(( err, result, ) => { // comment }); ``` which looks bad, instead this PR makes it break ```js SuperSuperSuperSuperSuperSuperSuperSuperSuperSuperSuperSuperLongCall( (err, result) => { // comment } ); ``` which look betterConfiguration menu - View commit details
-
Copy full SHA for 211e18b - Browse repository at this point
Copy the full SHA 211e18bView commit details -
Keep parenthesis on export default function (#844)
It turns out that my fix was not correct. We should not add parenthesis for FunctionDeclaration instead of checking if the function expression is named. Fixes #819
Configuration menu - View commit details
-
Copy full SHA for 3649835 - Browse repository at this point
Copy the full SHA 3649835View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5523c4f - Browse repository at this point
Copy the full SHA 5523c4fView commit details -
gofmt has this option and it's very handy for commit-hooks. Fixes #851
Configuration menu - View commit details
-
Copy full SHA for 205458a - Browse repository at this point
Copy the full SHA 205458aView commit details -
Add parenthesis around assignments (#862)
This is a neat trick from the React codebase. It helps highlight the fact that this is an assignment and not a comparison which is subtle to realize. Fixes #861
Configuration menu - View commit details
-
Copy full SHA for fa9dca3 - Browse repository at this point
Copy the full SHA fa9dca3View commit details -
Do not put \n after label (#860)
This is a leftover from the recast prototype, it hasn't been touched since then. I have never seen anyone not put the label on the same line. Fixes #859
Configuration menu - View commit details
-
Copy full SHA for 735f07f - Browse repository at this point
Copy the full SHA 735f07fView commit details -
Fix comment for
call( // comment(#858)This is a pretty exotic way to put a comment and it's been a long time since I have seen prettier output be as broken :) Fixes #857
Configuration menu - View commit details
-
Copy full SHA for fe38dab - Browse repository at this point
Copy the full SHA fe38dabView commit details -
Do not break long it calls (#842)
This happens very frequently that naming a test makes the entire line go > 80 columns and requires to indent everything which takes a lot more space. We've had this being reported multiple times and it also affects a lot of tests inside of fb. Fixes #159 (Note, this is built on-top of #841 but github doesn't handle stacked pull requests well...)
Configuration menu - View commit details
-
Copy full SHA for 184382d - Browse repository at this point
Copy the full SHA 184382dView commit details -
Fix flow union comments (#853)
The comments infra has been architected with trailing operators and fails for leading `|`. Instead of having leading comments, we can put trailing comments on the previous one and use the same technique as assignment to deal with the indentation. Fixes #849
Configuration menu - View commit details
-
Copy full SHA for 7d24b0a - Browse repository at this point
Copy the full SHA 7d24b0aView commit details -
Configuration menu - View commit details
-
Copy full SHA for b570154 - Browse repository at this point
Copy the full SHA b570154View commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff 0.20.0...0.21.0