Skip to content

Commit 7471a26

Browse files
authored
docs: update contributing instructions and add help command to makefile (#249)
1 parent 046ace7 commit 7471a26

File tree

2 files changed

+31
-11
lines changed

2 files changed

+31
-11
lines changed

CONTRIBUTING.md

Lines changed: 11 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -18,27 +18,28 @@ Before contributing to the `posit-sdk`, ensure that the following prerequisites
1818
> [!WARNING]
1919
> Executing `make` will utilize `pip` to install third-party packages in your activated Python environment. Please review the [`Makefile`](./Makefile) to verify behavior before executing any commands.
2020
21-
1. Fork the repository and clone it to your local machine.
22-
1. Create a new branch for your feature or bug fix.
23-
1. Run `make` to run the default development workflow.
24-
1. Make your changes and test them thoroughly using `make test`
25-
1. Run `make fmt` and `make lint` to verify adherence to the project style guide.
26-
1. Commit your changes and push them to your forked repository.
27-
1. Submit a pull request to the main repository.
21+
1. Fork the repository and open it in your development environment.
22+
2. Run `make` to run the default development workflow.
23+
3. Make your changes and test them thoroughly using `make test`
24+
4. Run `make fmt` and `make lint` to verify adherence to the project style guide.
25+
5. Commit your changes and push them to your forked repository.
26+
6. Submit a pull request to the main repository.
2827

2928
## Tooling
3029

3130
Use the default make target to execute the full build pipeline. For details on specific targets, refer to the [Makefile](./Makefile).
3231

3332
## Style Guide
3433

35-
We use [Ruff](https://docs.astral.sh/ruff/) for linting and code formatting. All proposed changes must successfully pass the `make lint` rules prior to merging.
34+
We use [Ruff](https://docs.astral.sh/ruff/) for linting and code formatting. Run `make deps` to install it.
3635

37-
Utilize `make fmt lint fix` to format and lint your changes.
36+
All proposed changes must successfully pass the `make lint` rules prior to merging.
37+
38+
Utilize `make fmt lint` to format and lint your changes.
3839

3940
### (Optional) pre-commit
4041

41-
This project is configured for [pre-commit](https://pre-commit.com). Once enabled, a `git commit` hook is created, which invokes `make fmt lint fix`.
42+
This project is configured for [pre-commit](https://pre-commit.com). Once enabled, a `git commit` hook is created, which invokes `make fmt lint`.
4243

4344
To enable pre-commit on your machine, run `pre-commit install`.
4445

Makefile

Lines changed: 20 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ include vars.mk
22

33
.DEFAULT_GOAL := all
44

5-
.PHONY: build clean cov default deps dev docs fmt fix install it lint test uninstall version
5+
.PHONY: build clean cov default deps dev docs fmt fix install it lint test uninstall version help
66

77
all: deps dev test lint build
88

@@ -60,3 +60,22 @@ uninstall:
6060

6161
version:
6262
@$(PYTHON) -m setuptools_scm
63+
64+
help:
65+
@echo "Makefile Targets"
66+
@echo " all Run deps, dev, test, lint, and build"
67+
@echo " build Build the project"
68+
@echo " clean Clean up project artifacts"
69+
@echo " cov Generate a coverage report"
70+
@echo " cov-html Generate an HTML coverage report and open it"
71+
@echo " cov-xml Generate an XML coverage report"
72+
@echo " deps Install dependencies"
73+
@echo " dev Install the project in editable mode"
74+
@echo " docs Build the documentation"
75+
@echo " fmt Format the code"
76+
@echo " install Install the built project"
77+
@echo " it Run integration tests"
78+
@echo " lint Lint the code"
79+
@echo " test Run unit tests with coverage"
80+
@echo " uninstall Uninstall the project"
81+
@echo " version Display the project version"

0 commit comments

Comments
 (0)