mvhutz/lambda
1
0
Fork 0
Code Issues 3 Pull Requests Actions Packages Projects Releases Wiki Activity

feat: add PHONY declarations and docs target to Makefile #9

Merged
mvhutz merged 1 commits from feat/makefile-improvements into main 2026-01-10 23:29:31 +00:00
Conversation 0 Commits 1 Files Changed 1
+6
mvhutz commented 2026-01-10 23:28:38 +00:00
Owner
Copy Link
Copy Source
name, about, title, ref, assignees, labels
name about title ref assignees labels
Default Template The default template for `lambda`. feat: add PHONY declarations and docs target to Makefile main

Description

This PR improves the Makefile by adding proper .PHONY declarations and a new docs target.
The .PHONY declarations ensure make properly handles targets that don't create files.
The docs target provides an easy way to view Go package documentation locally using godoc.

Changes:

  • Add .PHONY declaration for all non-file targets (it, run, profile, explain, graph, docs).
  • Add docs target that runs godoc server on port 6060 with helpful usage message.

Decisions

None.

Benefits

  • Prevents make from getting confused if files with target names exist.
  • Provides convenient documentation viewing with make docs.
  • Improves Makefile maintainability following best practices.

Checklist

  • Code follows conventional commit format.
  • Branch follows naming convention (<type>/<description>). Always use underscores.
  • Tests pass (if applicable).
  • Documentation updated (if applicable).
--- name: "Default Template" about: "The default template for `lambda`." title: "feat: add PHONY declarations and docs target to Makefile" ref: "main" assignees: [] labels: [] --- ## Description This PR improves the Makefile by adding proper `.PHONY` declarations and a new `docs` target. The `.PHONY` declarations ensure make properly handles targets that don't create files. The `docs` target provides an easy way to view Go package documentation locally using godoc. Changes: - Add `.PHONY` declaration for all non-file targets (`it`, `run`, `profile`, `explain`, `graph`, `docs`). - Add `docs` target that runs godoc server on port 6060 with helpful usage message. ### Decisions None. ## Benefits - Prevents make from getting confused if files with target names exist. - Provides convenient documentation viewing with `make docs`. - Improves Makefile maintainability following best practices. ## Checklist - [x] Code follows conventional commit format. - [x] Branch follows naming convention (`<type>/<description>`). Always use underscores. - [ ] Tests pass (if applicable). - [x] Documentation updated (if applicable).
mvhutz added 1 commit 2026-01-10 23:28:39 +00:00
feat: add PHONY declarations and docs target to Makefile ce4588e92d 
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
mvhutz merged commit 22019acbb1 into main 2026-01-10 23:29:31 +00:00
mvhutz deleted branch feat/makefile-improvements 2026-01-10 23:29:31 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
Compat/Breaking
Breaking change that won't be backward compatible
 Kind/Bug
Something is not working
 Kind/Documentation
Documentation changes
 Kind/Enhancement
Improve existing functionality
 Kind/Feature
New functionality
 Kind/Security
This is security issue
 Kind/Testing
Issue or pull request related to testing
Priority
Critical
1
The priority is critical
Priority
High
2
The priority is high
Priority
Low
4
The priority is low
Priority
Medium
3
The priority is medium
Reviewed
Confirmed
1
Issue has been confirmed
Reviewed
Duplicate
2
This issue or pull request already exists
Reviewed
Invalid
3
Invalid issue
Reviewed
Won't Fix
3
This issue won't be fixed
Status
Abandoned
3
Somebody has started to work on this but abandoned work
Status
Blocked
1
Something is blocking this issue or pull request
Status
Need More Info
2
Feedback is required to reproduce issue or to continue work
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
mvhutz (Maxim Hutz)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: mvhutz/lambda#9