Skip to content

change: merge code-style with style categories#1573

Merged
chavacava merged 2 commits intomgechev:masterfrom
alexandear:change/merge-code-style-to-style
Nov 9, 2025
Merged

change: merge code-style with style categories#1573
chavacava merged 2 commits intomgechev:masterfrom
alexandear:change/merge-code-style-to-style

Conversation

@alexandear
Copy link
Copy Markdown
Collaborator

I propose merging the code-style and style linter categories into one.

FailureCategoryBadPractice FailureCategory = "bad practice"
// FailureCategoryCodeStyle indicates code style issues.
//
// Deprecated: use FailureCategoryStyle instead.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This would be better in the documentation

Suggested change
// Deprecated: use FailureCategoryStyle instead.
// Deprecated: use [FailureCategoryStyle] instead.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

It doesn't matter for consts

pkgsite:

image

VS Code:

image

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

it looks like a bug to me in godoc then. But OK, thanks for checking.

My IDE follows such notation anyway, so I use it everywhere now.

@chavacava chavacava merged commit b73cca9 into mgechev:master Nov 9, 2025
10 checks passed
@alexandear alexandear deleted the change/merge-code-style-to-style branch November 9, 2025 19:53
@alexandear alexandear mentioned this pull request Nov 9, 2025
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