Skip to content

Add .pre-commit.config.yaml#176

Closed
dantti wants to merge 1 commit intolinuxdeploy:masterfrom
dantti:dantti/pre-commit
Closed

Add .pre-commit.config.yaml#176
dantti wants to merge 1 commit intolinuxdeploy:masterfrom
dantti:dantti/pre-commit

Conversation

@dantti
Copy link
Copy Markdown
Contributor

@dantti dantti commented Jun 13, 2024

Adding pre-commit support to the project to make
it easier for contributors to cope with code style.

You can also enable the pre-commit bot which will automatically push a commit to PR fixing it if there are issues.

Adding pre-commit support to the project to make
it easier for contributors to cope with code style.
@dantti
Copy link
Copy Markdown
Contributor Author

dantti commented Jun 13, 2024

lol I just noticed that there's actually no .clang-format, I was pretty sure I saw one.
I use this .clang-format, do you have one that you use?

@TheAssassin
Copy link
Copy Markdown
Member

I do not use any auto formatter. The code style is quite obvious, it's nothing fancy.

@TheAssassin
Copy link
Copy Markdown
Member

Thanks, but I personally see no use in this addition, as it's not part of my toolchain. I don't know anyone who'd be using this.

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.

3 participants