Skip to content

Fix E127 #102

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.

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

Merged
merged 1 commit into from
Jul 22, 2018
Merged

Fix E127 #102

merged 1 commit into from
Jul 22, 2018

Conversation

blueyed
Copy link
Member

@blueyed blueyed commented Jul 16, 2018

Fixes #100.

@blueyed blueyed mentioned this pull request Jul 16, 2018
@blueyed blueyed force-pushed the e127 branch 4 times, most recently from 9348831 to 7c700dd Compare July 21, 2018 20:11
@codecov
Copy link

codecov bot commented Jul 21, 2018

Codecov Report

Merging #102 into master will increase coverage by 0.5%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #102     +/-   ##
=========================================
+ Coverage   90.23%   90.74%   +0.5%     
=========================================
  Files           1        1             
  Lines         215      216      +1     
=========================================
+ Hits          194      196      +2     
+ Misses         21       20      -1
Impacted Files Coverage Δ
indent/python.vim 90.74% <100%> (+0.5%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c5e3545...5b4c03e. Read the comment docs.

> continuation line over-indented for visual indent

It appears that the "if (" is just the 4 columns wide that triggers the
extra indent with sw=4.

Might also change/adjust b:control_statement to ignore `if (`, but this
appears to be more flexible/correct.
@blueyed blueyed merged commit fef3759 into master Jul 22, 2018
@blueyed blueyed deleted the e127 branch July 22, 2018 21:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant