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

feat(null_ls_deps): add gofumpt and gopls #996

Merged
merged 2 commits into from
Sep 17, 2023

Conversation

cherrot
Copy link
Contributor

@cherrot cherrot commented Sep 13, 2023

  • Enable gofumpt in gopls setting since it's widely used in Go community
  • Import missing packages via goimports on save.

@Jint-lzxy Jint-lzxy changed the title feat(completion): use gofumpt to format Go code, run goimports on save feat(null_ls_deps): add gofumpt and gopls Sep 17, 2023
Copy link
Collaborator

@Jint-lzxy Jint-lzxy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🍯

@ayamir ayamir merged commit ce3639e into ayamir:main Sep 17, 2023
2 checks passed
singlemancombat pushed a commit to singlemancombat/nvim-config that referenced this pull request Sep 18, 2023
* feat(completion): use gofumpt to format Go code, run goimports on save

* chore(deps): sort null_ls deps

---------

Co-authored-by: Jint-lzxy <50296129+Jint-lzxy@users.noreply.github.com>
@Saafo
Copy link
Contributor

Saafo commented Sep 20, 2023

How to disable these two in user setting? Merging will not remove.

@CharlesChiuGit
Copy link
Collaborator

@Saafo take another look at https://github.com/ayamir/nvimdots/wiki/Usage#about-user-overrides-settings-options-events-etc

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.

5 participants