Skip to content

Conversation

dunglas
Copy link
Owner

@dunglas dunglas commented Jan 14, 2024

Super-Linter is a must-have; it automatically lints all files in the project, using popular linters.

It is especially useful to lint all the non-PHP files contained in a modern project: Dockerfile, YAML, Kubernetes manifests, GitHub Actions...

@dunglas dunglas force-pushed the feat/super-linter branch from 034b4ff to 69fd710 Compare August 29, 2025 13:42
@dunglas dunglas force-pushed the feat/super-linter branch from a14cce1 to f48c792 Compare August 29, 2025 15:18
@dunglas dunglas marked this pull request as ready for review August 29, 2025 15:20
@dunglas dunglas force-pushed the feat/super-linter branch from f48c792 to 92c8ef0 Compare August 29, 2025 15:28
@dunglas dunglas changed the title feat: lint the project with Super Linter feat: lint the whole project using Super-Linter Aug 31, 2025
dunglas and others added 2 commits September 1, 2025 12:31
Co-authored-by: Stanislau Kviatkouski <[email protected]>
@7-zete-7
Copy link
Contributor

7-zete-7 commented Sep 1, 2025

Does it make sense to keep only one of the code block lang (​​diff or patch)?

Copy link
Contributor

@7-zete-7 7-zete-7 left a comment

Choose a reason for hiding this comment

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

Keep empty column for non-changed lines in diff and patch code blocks

Co-authored-by: Stanislau Kviatkouski <[email protected]>
@dunglas
Copy link
Owner Author

dunglas commented Sep 2, 2025

@7-zete-7 yes indeed, consistency is always better

@dunglas
Copy link
Owner Author

dunglas commented Sep 2, 2025

According to https://stackoverflow.com/a/68222754, diff seems more accurate

@7-zete-7

This comment was marked as off-topic.

@dunglas
Copy link
Owner Author

dunglas commented Sep 2, 2025

Could you open another PR please?

@7-zete-7
Copy link
Contributor

7-zete-7 commented Sep 2, 2025

Yep, I'll open separated. Sorry for the annoying suggestions.

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