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: facebook/ktfmt
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: 02537ed
Choose a base ref
...
head repository: facebook/ktfmt
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 6ebfe03
Choose a head ref
  • 5 commits
  • 7 files changed
  • 4 contributors

Commits on Mar 9, 2022

  1. Bump version to 0.35-SNAPSHOT

    Reviewed By: strulovich
    
    Differential Revision: D34759565
    
    fbshipit-source-id: 3a8fcfc4606f6c1462680e0aaf85bbf4de7c7d25
    cgrushko authored and facebook-github-bot committed Mar 9, 2022
    Configuration menu
    Copy the full SHA
    2ada6d2 View commit details
    Browse the repository at this point in the history

Commits on Mar 16, 2022

  1. Fix .editorconfig-kotlinglang (#299)

    Summary:
    `ktfmt` with the `--kotlinlang-style` option actually uses a continuation indent size of 4, but the editorconfig provided in this incorrectly configures a continuation indent of 8.
    
    Pull Request resolved: #299
    
    Reviewed By: strulovich
    
    Differential Revision: D34890733
    
    Pulled By: cgrushko
    
    fbshipit-source-id: d020a35b9e3c82b201e27282e484974d30b56579
    avarun42 authored and facebook-github-bot committed Mar 16, 2022
    Configuration menu
    Copy the full SHA
    f3c2964 View commit details
    Browse the repository at this point in the history

Commits on Mar 21, 2022

  1. Always preserve semicolons preceding lambda expressions, even when th…

    …ey might be meaningless (#301)
    
    Summary:
    There are a huge number of cases where removal might change behaviour, because the trailing
    lambda syntax is so flexible. Therefore, we just assume that all semicolons followed by
    lambdas are meaningful. The cases where they could be removed are too rare to justify the
    risk of changing behaviour.
    
    #278 tried to fix this earlier, but missed
    a lot of cases. Some examples are added to the tests.
    
    Pull Request resolved: #301
    
    Reviewed By: strulovich
    
    Differential Revision: D34999205
    
    Pulled By: cgrushko
    
    fbshipit-source-id: 7f1bc75ba450e1e98d7b764f93bc613188dfa5cd
    nreid260 authored and facebook-github-bot committed Mar 21, 2022
    Configuration menu
    Copy the full SHA
    a66fbc4 View commit details
    Browse the repository at this point in the history

Commits on Mar 22, 2022

  1. Allow breaking before . in a reciever function declaration to avoid…

    … long lines
    
    Summary:
    This fixes:
    
    #268
    
    It fixes it for function only, but I can try to extend into the declareOne method later (it's just more complicated for that)
    
    Reviewed By: cgrushko
    
    Differential Revision: D35047687
    
    fbshipit-source-id: afe6411110becdfa49d14e57cfdfbe19f4d9de87
    strulovich authored and facebook-github-bot committed Mar 22, 2022
    Configuration menu
    Copy the full SHA
    f0c6281 View commit details
    Browse the repository at this point in the history

Commits on Mar 23, 2022

  1. Bump version to 0.35

    Reviewed By: strulovich
    
    Differential Revision: D35076765
    
    fbshipit-source-id: 4619ad16fe24f4e0afecda48fdc792e63f87daab
    cgrushko authored and facebook-github-bot committed Mar 23, 2022
    Configuration menu
    Copy the full SHA
    6ebfe03 View commit details
    Browse the repository at this point in the history
Loading