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

QoL: Code formatting config for better consistency? #21

Closed
ducktordanny opened this issue Jul 26, 2024 · 1 comment · Fixed by #33
Closed

QoL: Code formatting config for better consistency? #21

ducktordanny opened this issue Jul 26, 2024 · 1 comment · Fixed by #33
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@ducktordanny
Copy link
Contributor

ducktordanny commented Jul 26, 2024

For easier contribution and better consistency, I'd suggest adding some type of formatting config, e.g. in my configs, I use StyLua which only requires a .stylua.toml file with your preferences.

After this, you can use e.g. Mason or whatever to get the LSP for it and have any formatting plugin to apply the proper formatting (e.g. I use conform.nvim).

Any thoughts on this? I think it would be great to have something like this :)

(Edit: I wanted to post this in a discussion but clicked the button in the wrong tab... I'll leave it here now, hope it's fine)

@p5quared p5quared added enhancement New feature or request good first issue Good for newcomers labels Jul 26, 2024
@p5quared
Copy link
Owner

A little formatting definitely goes a long way. I think it's lower priority than some of the 'front-facing' stuff like #20 or #15 but worth implementing at some point.

Thanks for sharing your configs for reference, I'll take a look!

@p5quared p5quared linked a pull request Sep 8, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants