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

fix: exclusions are no longer working as before #1237

Merged
merged 3 commits into from
Feb 13, 2025
Merged

Conversation

chavacava
Copy link
Collaborator

Fixes #1230 and #1233 by reverting #1060

Copy link
Collaborator

@denisvmedia denisvmedia left a comment

Choose a reason for hiding this comment

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

We have some weird test failing issue though 🤔

@chavacava
Copy link
Collaborator Author

We have some weird test failing issue though 🤔

go 1.24 changed the build info and now the version has a " (devel)" ... thus tests passing in 1.23 do not pass in 1.24 and the other way around. I'll fix

@chavacava chavacava merged commit 4020f43 into master Feb 13, 2025
7 checks passed
@denisvmedia denisvmedia deleted the revert/1060 branch February 13, 2025 16:06
@mfederowicz
Copy link
Contributor

sorry for that @chavacava maybe i should back to the drawing board with this issue (make more tests, especially with magic ./... patterns). Maybe good point will be adding some kind of switch in config (by default classic way, with other value new way with regexp), but i need more time for tests :)

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Exclusions broken
3 participants