forked from phoenix-oss/llama-stack-mirror
docs: propose new contribution guidance (#1750)
# What does this PR do? Propose new contribution guidance. Signed-off-by: Sébastien Han <seb@redhat.com>
This commit is contained in:
parent
00917ef5b2
commit
636d97207f
1 changed files with 4 additions and 2 deletions
|
@ -135,9 +135,11 @@ uv sync
|
||||||
|
|
||||||
## Coding Style
|
## Coding Style
|
||||||
|
|
||||||
|
* Comments should provide meaningful insights into the code. Avoid filler comments that simply describe the next step, as they create unnecessary clutter, same goes for docstrings.
|
||||||
|
* Prefer comments to clarify surprising behavior and/or relationships between parts of the code rather than explain what the next line of code does.
|
||||||
|
* Catching exceptions, prefer using a specific exception type rather than a broad catch-all like `Exception`.
|
||||||
|
* Error messages should be prefixed with "Failed to ..."
|
||||||
* 4 spaces for indentation rather than tabs
|
* 4 spaces for indentation rather than tabs
|
||||||
* 80 character line length
|
|
||||||
* ...
|
|
||||||
|
|
||||||
## Common Tasks
|
## Common Tasks
|
||||||
|
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue