-
-
Notifications
You must be signed in to change notification settings - Fork 260
Re-enable checkmake #5817
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
base: main
Are you sure you want to change the base?
Re-enable checkmake #5817
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR re-enables the checkmake linter for Makefiles by removing the disabled configuration that was previously in place due to security concerns. The changes restore checkmake functionality across all MegaLinter flavors and deployment workflows.
- Removes the
disabled: true
flag and security warning from the makefile descriptor - Adds checkmake back to all flavor configurations and Dockerfiles
- Updates deployment workflows to include checkmake in the build matrix
Reviewed Changes
Copilot reviewed 24 out of 24 changed files in this pull request and generated no comments.
Show a summary per file
File | Description |
---|---|
megalinter/descriptors/makefile.megalinter-descriptor.yml | Removes disabled flag and security warning for checkmake |
megalinter/descriptors/all_flavors.json | Adds MAKEFILE_CHECKMAKE to all flavor configurations |
flavors/*/flavor.json | Adds checkmake to individual flavor linter lists |
flavors/*/Dockerfile | Adds checkmake Docker image and binary installation |
Dockerfile | Adds checkmake to main MegaLinter Docker image |
.github/workflows/deploy-*.yml | Adds checkmake to deployment workflow matrices |
✅
|
No description provided.