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

prepare-for-packaging: Explicitly mention linting #602

Merged
merged 1 commit into from
Feb 15, 2025

Conversation

ermo
Copy link
Contributor

@ermo ermo commented Feb 15, 2025

Description

@JoshStrobl was kind enough to report that we don't actually mention the word "linting" as he was going through the process of setting up a solus development system.

This commit fixes that and adds slightly more information on when and how linting happens in practice.

@ermo ermo force-pushed the mention-linting-with-git-hooks branch from f96521d to 5ef2be9 Compare February 15, 2025 19:47
@davidjharder
Copy link
Member

davidjharder commented Feb 15, 2025

Edit: D'oh it's flagging "liniting". Please fix

@JoshStrobl was kind enough to report that we don't actually mention the
word "linting" as he was going through the process of setting up a solus
development system.

This commit fixes that and adds slightly more information on when and how
linting happens in practice.

Signed-off-by: Rune Morling <[email protected]>
@ermo ermo force-pushed the mention-linting-with-git-hooks branch from 5ef2be9 to 642788b Compare February 15, 2025 22:16
Copy link
Member

@davidjharder davidjharder left a comment

Choose a reason for hiding this comment

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

Thanks!

@davidjharder davidjharder merged commit 73249a8 into master Feb 15, 2025
1 check passed
@davidjharder davidjharder deleted the mention-linting-with-git-hooks branch February 15, 2025 22:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Completed
Development

Successfully merging this pull request may close these issues.

2 participants