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.

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

Added autocmd to update status-line #1000

Merged
merged 1 commit into from Nov 15, 2017
Merged

Conversation

@sei40kr
Copy link
Contributor

@sei40kr sei40kr commented Nov 14, 2017

Added autocmd to update status-line

Added autocmd to update status-line on ALE process completion.

This is still not perfect fix.
It needs to detect to the target buffer is currently active or not.

Added autocmd to update status-line on ALE process completion.
@codecov
Copy link

@codecov codecov bot commented Nov 14, 2017

Codecov Report

Merging #1000 into dev will increase coverage by 0.02%.
The diff coverage is 80%.

@@            Coverage Diff             @@
##              dev    #1000      +/-   ##
==========================================
+ Coverage   33.75%   33.78%   +0.02%     
==========================================
  Files         218      218              
  Lines        9293     9298       +5     
==========================================
+ Hits         3137     3141       +4     
- Misses       6156     6157       +1
@sei40kr
Copy link
Contributor Author

@sei40kr sei40kr commented Nov 15, 2017

In short, "Fixed an issue that statusline won't be updated after ALE process"

@wsdjeg
Copy link
Member

@wsdjeg wsdjeg commented Nov 15, 2017

Thanks.

@wsdjeg wsdjeg merged commit b1d6c15 into SpaceVim:dev Nov 15, 2017
5 checks passed
5 checks passed
@codecov
codecov/patch 80% of diff hit (target 33.75%)
Details
@codecov
codecov/project 33.78% (+0.02%) compared to f2a8ef0
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
license/cla Contributor License Agreement is signed.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants