Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

fix: fixes #1103 by fixing the processing of max-public-structs arguments #1114

Merged
merged 2 commits into from
Nov 15, 2024
Merged

fix: fixes #1103 by fixing the processing of max-public-structs arguments #1114

merged 2 commits into from
Nov 15, 2024

Conversation

chavacava
Copy link
Collaborator

Closes #1103

ccoVeille
ccoVeille suggested changes Nov 12, 2024
View reviewed changes
Copy link
Contributor

@ccoVeille ccoVeille left a comment

Choose a reason for hiding this comment

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

Typo in commit message and PR title

plublic => public

@ccoVeille ccoVeille mentioned this pull request Nov 12, 2024
Closed
ccoVeille
ccoVeille reviewed Nov 12, 2024
View reviewed changes
@denisvmedia denisvmedia changed the title fix: fixes #1103 by fixing the processing of max-plublic-structs arguments fix: fixes #1103 by fixing the processing of max-public-structs arguments Nov 13, 2024
@chavacava chavacava merged commit 7ee4500 into master Nov 15, 2024
4 checks passed
@chavacava chavacava deleted the fix/1103 branch February 4, 2025 11:06
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@ccoVeille ccoVeille ccoVeille requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

The max-public-structs rule starts finding many false positives in version 1.5.0
2 participants
@chavacava @ccoVeille