Skip to content

RFC - Test modules with issues regarding strict and warnings#24709

Draft
perlpunk wants to merge 29 commits intoos-autoinst:masterfrom
perlpunk:strict-warnings-issues
Draft

RFC - Test modules with issues regarding strict and warnings#24709
perlpunk wants to merge 29 commits intoos-autoinst:masterfrom
perlpunk:strict-warnings-issues

Conversation

@perlpunk
Copy link
Contributor

@perlpunk perlpunk commented Feb 20, 2026

🧹 I'm building a tool to run test modules with strict and warnings and check if they compile. I want to add it as a github action at some point.

⚠️ IMPORTANT: The commits I made are in most cases just cheap fixes to make strict happy. They will need a proper fix instead.

👓 Please look through the commits and see if you can fix some of them. I will then continuously rebase this PR until hopefully all issues are solved.

  • This is just a first step to get rid of issues with strict.
  • There will be more issues coming regarding warnings.
  • Note that I think these are not often only issues with strict, but I think those tests cannot work correctly.
    In the future when we enforce strict and warnings, we can avoid such broken tests.

Thanks! 🙏

@mpagot
Copy link
Contributor

mpagot commented Feb 20, 2026

#24710

@perlpunk perlpunk force-pushed the strict-warnings-issues branch from ccc5f05 to d19c1ea Compare February 25, 2026 15:36
perlpunk added 25 commits March 10, 2026 23:11
@perlpunk perlpunk force-pushed the strict-warnings-issues branch from d19c1ea to 09d5b6f Compare March 10, 2026 22:14
@perlpunk
Copy link
Contributor Author

So I was able to remove some commits by rebasing, but unfortunately I had to add more commits with new errors.
I'll try to provide a script soon that you can use to check locally.

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