Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: prettier/prettier
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: 0.19.0
Choose a base ref
...
head repository: prettier/prettier
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 0.20.0
Choose a head ref
  • 20 commits
  • 56 files changed
  • 6 contributors

Commits on Feb 23, 2017

  1. Update docs to 0.19.0

    vjeux committed Feb 23, 2017
    Configuration menu
    Copy the full SHA
    71a1ca8 View commit details
    Browse the repository at this point in the history
  2. Run prettier on 0.19.0 (#793)

    vjeux authored Feb 23, 2017
    Configuration menu
    Copy the full SHA
    7c4654b View commit details
    Browse the repository at this point in the history
  3. Fix extra parens for update expressions (#796)

    * Remove parens around UpdateExpression when parent is CallExpression.
    
    * Add new test case.
    yamafaktory authored and vjeux committed Feb 23, 2017
    Configuration menu
    Copy the full SHA
    c8c1875 View commit details
    Browse the repository at this point in the history

Commits on Feb 24, 2017

  1. Fix empty options (#803)

    ```js
    // test.js
    var prettier = require('./');
    prettier.format('a=1');
    ```
    
    ```js
    node test.js
    // doesn't crash
    ```
    vjeux authored and jlongster committed Feb 24, 2017
    Configuration menu
    Copy the full SHA
    2111f11 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    cea2548 View commit details
    Browse the repository at this point in the history
  3. Fix function declaration params comments (#802)

    * Add new handleFunctionDeclarationComments function.
    
    * Add dangling comments printing for function params.
    
    * Add new test case.
    
    * Update tests.
    
    * Refactor handleFunctionDeclarationComments to only addDanglingComment when no params.
    
    * Remove unecessary helper function, only attach dangling comments when no params.
    
    * Reset flow tests, no more regression.
    yamafaktory authored and vjeux committed Feb 24, 2017
    Configuration menu
    Copy the full SHA
    b32ace8 View commit details
    Browse the repository at this point in the history
  4. Update flow to 0.40 (#808)

    This fixes the bug with comments swapped between `/*` and `//`
    
    Fixes #755
    vjeux authored Feb 24, 2017
    Configuration menu
    Copy the full SHA
    1b9234a View commit details
    Browse the repository at this point in the history
  5. Correct link for travis

    vjeux authored Feb 24, 2017
    Configuration menu
    Copy the full SHA
    9a447c0 View commit details
    Browse the repository at this point in the history

Commits on Feb 25, 2017

  1. Fix function call args (#809)

    * Implement new heuristic for arguments in printArgumentsList function.
    
    * Remove unwanted modification.
    
    * Add new helper function.
    
    * Update tests according.
    
    * Add new tests for #682, #677, #253.
    
    * Move heuristic in groupLastArg condition.
    
    * Update tests.
    yamafaktory authored and vjeux committed Feb 25, 2017
    Configuration menu
    Copy the full SHA
    04f09dd View commit details
    Browse the repository at this point in the history

Commits on Feb 27, 2017

  1. Properly support do (#811)

    The code that supported `do` was likely from some recast time and didn't work anymore.
    vjeux authored and jlongster committed Feb 27, 2017
    Configuration menu
    Copy the full SHA
    443a5d6 View commit details
    Browse the repository at this point in the history
  2. Do not put parenthesis around not named default export (#819)

    We need to add parenthesis around function expressions if they are named otherwise the name leak, but if the function is not named then it's just superfluous.
    
    The conditional change is due to a bad use of switch case where it would fall through the next one. We don't want parenthesis there.
    vjeux authored and jlongster committed Feb 27, 2017
    Configuration menu
    Copy the full SHA
    08b8a5f View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    a59e2c6 View commit details
    Browse the repository at this point in the history
  4. Fix trailing commas in docs (#825)

    I also added support for parser dropdown and stop minifying the bundle so that the flow parser actually works.
    
    Fixes #816
    vjeux authored and jlongster committed Feb 27, 2017
    Configuration menu
    Copy the full SHA
    edf2bde View commit details
    Browse the repository at this point in the history
  5. Put short body of arrow functions on the same line (#829)

    By adding a group, we can avoid adding a newline if the expression fits in a single line.
    
    Fixes #800
    vjeux authored and jlongster committed Feb 27, 2017
    Configuration menu
    Copy the full SHA
    4540f47 View commit details
    Browse the repository at this point in the history

Commits on Feb 28, 2017

  1. Preserve new lines for comments after = (#830)

    The `hasLeadingOwnLineComment` helper is going to be useful for other places where we just used `n.comments` as a proxy for it.
    
    Fixes #660
    vjeux authored and jlongster committed Feb 28, 2017
    Configuration menu
    Copy the full SHA
    c17bcab View commit details
    Browse the repository at this point in the history
  2. Fix indentation of a merged group (#828)

    Printing a merged group indented was actually not the right fix. The right fix was to print them in a single line. It used to have this behavior when I was mutating the first group but now that I don't anymore I need to reproduce this condition.
    
    Fixes #823
    vjeux authored and jlongster committed Feb 28, 2017
    Configuration menu
    Copy the full SHA
    2ea1dbc View commit details
    Browse the repository at this point in the history
  3. Migrate class comments to the beginning (#831)

    Instead of trying to figure out a complicated way to preserve their correct position, it's easier to just migrate those comments before the class.
    
    Fixes #693
    Fixes #694
    vjeux authored and jlongster committed Feb 28, 2017
    Configuration menu
    Copy the full SHA
    6148266 View commit details
    Browse the repository at this point in the history
  4. Update list of related projects (#833)

    lydell authored and vjeux committed Feb 28, 2017
    Configuration menu
    Copy the full SHA
    2534067 View commit details
    Browse the repository at this point in the history
  5. Allow breaking for logical expressions in member chains (#827)

    I'm not really sure what a general rule is for those, but starting with LogicalExpressions should be good. Outside of identifiers, function calls and logical expressions, there aren't a lot of things you'd put there in real code anyway.
    
    Fixes #822
    vjeux authored Feb 28, 2017
    Configuration menu
    Copy the full SHA
    d6e3815 View commit details
    Browse the repository at this point in the history
  6. 0.20.0

    vjeux committed Feb 28, 2017
    Configuration menu
    Copy the full SHA
    e09bdd4 View commit details
    Browse the repository at this point in the history
Loading