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

lint: homogenize code quality tool calls #2727

Merged
merged 2 commits into from
Feb 20, 2020
Merged

Conversation

bhcleek
Copy link
Collaborator

@bhcleek bhcleek commented Feb 16, 2020

lint: homogenize vet and errcheck handling
  • Use go#util#ExecInDir() when running go vet.
  • Make the progress message when starting go vet consistent with
    errcheck's similiar progress message.
  • Call go#cmd#autowrite() when starting go#lint#Errcheck to either write
    the buffers that have modifications when 'autowrite' or 'autowriteall'
    are set or display a message if neither of those options are set,
    because errcheck only analyzes files on disk.
  • chagne the error message that's displayed when the package name cannot
    be determined to avoid implying that the package must be in GOPATH
    when the user is operating in module aware mode.
  • Do not echo the start of errcheck progress if g:go_echo_command_info
    is cleared.
  • disable g:go_echo_command_info when running tests to avoid screen
    clutter when tests fail.
lint: set progress and statusline for consistently

* Use go#util#ExecInDir() when running go vet.
* Make the progress message when starting go vet consistent with
  errcheck's similiar progress message.
* Call go#cmd#autowrite() when starting go#lint#Errcheck to either write
  the buffers that have modifications when 'autowrite' or 'autowriteall'
  are set or display a message if neither of those options are set,
  because errcheck only analyzes files on disk.
* chagne the error message that's displayed when the package name cannot
  be determined to avoid implying that the package must be in GOPATH
  when the user is operating in module aware mode.
* Do not echo the start of errcheck progress if g:go_echo_command_info
  is cleared.
* disable g:go_echo_command_info when running tests to avoid screen
  clutter when tests fail.
@bhcleek bhcleek merged commit a32f714 into fatih:master Feb 20, 2020
@bhcleek bhcleek deleted the lint/homogenize branch February 20, 2020 18:48
bhcleek added a commit that referenced this pull request Feb 20, 2020
@bhcleek bhcleek added this to the vim-go 1.23 milestone Feb 26, 2020
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