Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix: nested indenting for offsetTernaryExpressions: true (fixes #13971) #13972

Merged
merged 3 commits into from Jan 30, 2021
Merged

Fix: nested indenting for offsetTernaryExpressions: true (fixes #13971) #13972

merged 3 commits into from Jan 30, 2021

Commits on Jan 29, 2021

  1. reproduce extra nested ternary indentation with a test

    @brodybits
    brodybits committed Jan 29, 2021
    Copy the full SHA
    1dc0f0f View commit details
    Browse the repository at this point in the history

  2. add a similar test with proper indenting of nested ternary

    @brodybits
    brodybits committed Jan 29, 2021
    Copy the full SHA
    266a06d View commit details
    Browse the repository at this point in the history

  3. apply condition to avoid extra indenting with offsetTernaryExpressions 

    Co-authored-by: Adam Stankiewicz <sheerun@sher.pl>
    @brodybits @sheerun
    brodybits and sheerun committed Jan 29, 2021
    Copy the full SHA
    8ed37bb View commit details
    Browse the repository at this point in the history