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

Add a post-completion hook to ALE #3333

Closed
w0rp opened this issue Sep 4, 2020 · 1 comment
Closed

Add a post-completion hook to ALE #3333

w0rp opened this issue Sep 4, 2020 · 1 comment
Labels
enhancement
Projects
On the Radar

Comments

@w0rp
Copy link
Member

w0rp commented Sep 4, 2020

In order to set up executing any Vim code at all, and especially for calling commands like ALEFix, we should add a post-completion hook for ALE completion like ALECompletePost, much like the existing linting hooks.
@w0rp w0rp added this to To Do in On the Radar via automation Sep 4, 2020
@w0rp w0rp moved this from To Do to In Progress in On the Radar Sep 4, 2020
@w0rp w0rp mentioned this issue Sep 6, 2020
Closed
@w0rp w0rp closed this as completed in 7d90ff5 Sep 8, 2020
On the Radar automation moved this from In Progress to Done Sep 8, 2020
@w0rp
Copy link
Member Author

w0rp commented Sep 8, 2020

I have implemented this now along with a some other command changes, so it can be used to fix code after you insert completions, or :ALEImport. See :help ALECompletePost.

jsit added a commit to jsit/ale that referenced this issue Apr 19, 2021
@jsit
Merge branch 'master' into fix-3271
b2ab776 
* master: (123 commits)
  Add tests for maven.vim file
  Fix grammatical error in doc
  Add maven helper file; use maven wrapper if available instead of global 'mvn' executable
  fix lint, fix variable semantics and update tests
  bibclean: update matchlist reges for bibclean > v2.11.4
  Bump the ALE version to 3.0.0
  Close dense-analysis#2522 - Check pylint on the fly
  Add tests for covering the coming Vint version
  Close dense-analysis#3003 - Show ignored linters in :ALEInfo
  Close dense-analysis#3333 - Add an ALECompletePost event
  Fix a completion error
  Fix typo
  Close dense-analysis#3268 - Implement :ALEImport
  Fix dense-analysis#3183 - Escape filename characters from LSP/tsserver
  Clean up embertemplatelint code
  Fix asciidoc languagetool integration
  Fix dense-analysis#3322 - Apply rename changes correctly
  feat(template-lint): Read from stdin (dense-analysis#2622)
  Fix flake8 cd logic for invalid options
  Add tests for \r removal
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
On the Radar
Done
Milestone
No milestone
Development

No branches or pull requests
1 participant
@w0rp