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

languages: refactor step 1 #593

Closed
wants to merge 8 commits into from
Closed

languages: refactor step 1 #593

wants to merge 8 commits into from

Conversation

NotAShelf
Copy link
Owner

@NotAShelf NotAShelf commented Feb 1, 2025

This is the first step to the dreaded language refactor. Adds necessary groundwork for refactoring the languages.* option modules latter on.

We would like to rely on conform-nvim and nvim-lint for modular and customizable formatting and linting, and then deprecate none-ls in the second step.

Goals:

  • User configurable formatters and linters
  • Decent interfaces to allow for internal usage

Things to do:

  • Decide on an interface for configuring formatters
  • Document new formatters and linters

Things done:

  • Add conform.nvim
  • Add nvim-lint (WIP)

github-actions bot pushed a commit that referenced this pull request Feb 1, 2025
Copy link

github-actions bot commented Feb 1, 2025

🚀 Live preview deployed from 4196be3

View it here:

Debug Information

Triggered by: NotAShelf

HEAD at: language-refactor-step1

Reruns: 262

This should allow language modules to define new formatters without accessing conform's setupOpts each time.
github-actions bot pushed a commit that referenced this pull request Feb 6, 2025
@NotAShelf
Copy link
Owner Author

Closing, as changes from this PR already made it into main branch.

@NotAShelf NotAShelf closed this Feb 16, 2025
Copy link

✅ Preview has been deleted successfully!

github-actions bot pushed a commit that referenced this pull request Feb 16, 2025
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.

2 participants