added automated "make help" target to make makefile commands discoverable #4091
added automated "make help" target to make makefile commands discoverable #4091arkid15r merged 4 commits intoOWASP:mainfrom
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
Summary by CodeRabbit
WalkthroughAdds a repository-wide automated Makefile help/default goal and reorganizes Makefile targets (root, backend, frontend, cspell, infrastructure). Updates CONTRIBUTING.md with a Getting Started note referencing make help. Changes are documentation and Makefile target additions/reformatting only. No runtime code changes. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
d7bea91
|
arkid15r
left a comment
There was a problem hiding this comment.
Looks good, we can improve grouping and some targets visibility (not all needed for contributors).
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #4091 +/- ##
=======================================
Coverage 99.71% 99.71%
=======================================
Files 519 519
Lines 16068 16068
Branches 2192 2192
=======================================
Hits 16022 16022
Misses 38 38
Partials 8 8
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|



Proposed change
Resolves #4080
The problem was people were confused when it came to running make commands like :
So to fix all this :
How it works, now :

Checklist
make check-testlocally: all warnings addressed, tests passed