Skip to content

Commit

Permalink
build(tools): use a single name for targets
Browse files Browse the repository at this point in the history
When 2 targets are define on the same line in the Makefile, the `make help` command does not properly parse the line and display the ##- prefix.

Limit ourselves to a single name per target to keep the help generation
fairly simple.

Signed-off-by: Paul Mars <[email protected]>
  • Loading branch information
upils committed Dec 4, 2024
1 parent c0520f5 commit 1d11d5f
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions common.mk
Original file line number Diff line number Diff line change
Expand Up @@ -118,7 +118,7 @@ lint-docs: ##- Lint the documentation
uv run --extra docs sphinx-lint --max-line-length 88 --enable all $(DOCS)

.PHONY: lint-twine
lint-twine: dist/* ##- Lint Python packages with twine
lint-twine: pack-pip ##- Lint Python packages with twine
uv tool run twine check dist/*

.PHONY: test
Expand Down Expand Up @@ -149,7 +149,7 @@ docs-auto: ## Build and host docs with sphinx-autobuild
uv run --extra docs sphinx-autobuild -b html --open-browser --port=8080 --watch $(PROJECT) -W $(DOCS) $(DOCS)/_build

.PHONY: pack-pip
pack-pip dist/*: ##- Build packages for pip (sdist, wheel)
pack-pip: ##- Build packages for pip (sdist, wheel)
uv build .

# Below are intermediate targets for setup. They are not included in help as they should
Expand Down

0 comments on commit 1d11d5f

Please sign in to comment.