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.

Sign up for GitHub

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

Jump to bottom

DevOps: Linting #55

Open
JakobLichterfeld opened this issue Dec 4, 2022 · 2 comments · May be fixed by #78
Open

DevOps: Linting #55

JakobLichterfeld opened this issue Dec 4, 2022 · 2 comments · May be fixed by #78
Assignees
@JakobLichterfeld
Labels
enhancement New feature or request

Comments

@JakobLichterfeld
Copy link
Collaborator

As a developer, the pushed code shut be linted automatically.
@JakobLichterfeld JakobLichterfeld added the enhancement New feature or request label Dec 5, 2022
@JakobLichterfeld
Copy link
Collaborator Author

We can even use Super-Linter to lint all used languages

@TheRealKasumi
Copy link
Owner

Sorry, I forgot to reply. I think I like the idea. Especially since its using clang-format for the C/C++ formatting which I personally like.

JakobLichterfeld added a commit that referenced this issue Dec 28, 2022
@JakobLichterfeld
DevOps: Linting: initial commit for Super-Linter on all branches and …
95eac6b 
…all pull-requests, #55
JakobLichterfeld added a commit that referenced this issue Dec 28, 2022
@JakobLichterfeld
DevOps: Linting: badge in readme, #55
95ad9b7
@JakobLichterfeld JakobLichterfeld linked a pull request Dec 28, 2022 that will close this issue
Open
@JakobLichterfeld JakobLichterfeld self-assigned this Dec 28, 2022
JakobLichterfeld added a commit that referenced this issue Dec 28, 2022
@JakobLichterfeld
DevOps: Linting: update action to be Node.js 16 compatible,see: https…
cc4bfe9 
…://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/ , #55
JakobLichterfeld added a commit that referenced this issue Dec 28, 2022
@JakobLichterfeld
DevOps: Linting: Super-Linter on all branches and all pull-requests, …
4036d8a 
…correct syntax, #55
JakobLichterfeld added a commit that referenced this issue Dec 28, 2022
@JakobLichterfeld
DevOps: Linting: badge in latest readme, #55
3a51924
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JakobLichterfeld JakobLichterfeld

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

DevOps: Linting TheRealKasumi/NikoLight
2 participants
@JakobLichterfeld @TheRealKasumi