Skip to content

Commit cc40d19

Browse files
Merge pull request #86 from python-accelerator-middle-layer/update-readme-contributing
Updated CONTRIBUTING and README
2 parents 9c3e1b0 + 65597b8 commit cc40d19

File tree

2 files changed

+12
-2
lines changed

2 files changed

+12
-2
lines changed

CONTRIBUTING.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,9 @@ Thank you for your interest in contributing to [PyAML]! This document outlines h
1717
### 3. Submitting Changes
1818
- **Fork & Branch:** Fork the repo and create a branch for your changes.
1919
- **Commits:** Write clear, atomic commit messages. Reference the issue number if applicable.
20+
- **Formatting:** Ensure your code adheres to the project’s style guidelines. Run tests locally before submitting. It is recommended to use `pre-commit` hooks for this purpose.
21+
Sometimes pre-commit hook will block you committing due to formatting issues. They may be fixed automatically or require manual intervention. If they are fixed automatically just re-stage the files and commit again.
22+
If manual intervention is required, please follow the instructions provided by the pre-commit tool.
2023
- **Pull Requests (PRs):**
2124
- Open a PR against the `main` branch.
2225
- Include a description of your changes and reference the relevant issue.

README.md

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,10 +15,17 @@ Disclaimer: the pyAML software is still under development.
1515
cd pyaml
1616
pip install -e .
1717
```
18-
4. If you want to try the examples using the TANGO bindings you also need [tango-pyaml](https://github.com/python-accelerator-middle-layer/tango-pyaml).
18+
19+
4. For developers: install the development dependencies and pre-commit hooks
20+
21+
```
22+
pip install -e .[dev]
23+
pre-commit install
24+
```
25+
5. If you want to try the examples using the TANGO bindings you also need [tango-pyaml](https://github.com/python-accelerator-middle-layer/tango-pyaml).
1926
Clone that repository and install the package inside the same virtual environment as the `pyaml` package.
2027
tango-pyaml will automatically install pyaml, so step 3 can be skipped.
21-
5. For tests, you may want to install dummy-cs/tango available in
28+
6. For tests, you may want to install dummy-cs/tango available in
2229
tests/dummy-cs/tango
2330

2431
#### Documentation

0 commit comments

Comments
 (0)