-
-
Notifications
You must be signed in to change notification settings - Fork 227
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Showing
31 changed files
with
367 additions
and
771 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,10 +1,10 @@ | ||
repos: | ||
- repo: https://github.com/ambv/black | ||
rev: 24.1.1 | ||
- repo: https://github.com/psf/black-pre-commit-mirror | ||
rev: 24.10.0 | ||
hooks: | ||
- id: black | ||
- repo: https://github.com/charliermarsh/ruff-pre-commit | ||
# Ruff version. | ||
rev: v0.2.0 | ||
- repo: https://github.com/astral-sh/ruff-pre-commit | ||
rev: v0.7.4 | ||
hooks: | ||
- id: ruff | ||
- id: ruff | ||
args: [--fix] |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,20 @@ | ||
{ | ||
"recommendations": [ | ||
"benspaulding.python-manifest-template", | ||
"charliermarsh.ruff", | ||
"esbenp.prettier-vscode", | ||
"fill-labs.dependi", | ||
"github.vscode-github-actions", | ||
"lextudio.restructuredtext", | ||
"ms-python.black-formatter", | ||
"ms-python.debugpy", | ||
"ms-python.python", | ||
"ms-python.vscode-pylance", | ||
"njpwerner.autodocstring", | ||
"redhat.vscode-yaml", | ||
"swyddfa.esbonio", | ||
"tamasfe.even-better-toml", | ||
"trond-snekvik.simple-rst", | ||
"twixes.pypi-assistant" | ||
] | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,31 +1,46 @@ | ||
# The basics | ||
# Contributing | ||
|
||
## The basics | ||
|
||
Your help is appreciated and welcome! | ||
|
||
The _master_ branch is meant to hold the release code. At any time this should be | ||
identical to the code available on PyPI. | ||
The _master_ branch is meant to hold the release code. At any time this should be | ||
identical to the code available on PyPI. | ||
|
||
PR's will be pushed on the _development_ branch if the actual package code is changed. When the time comes this branch | ||
will be merged to the _master_ branch and a new release will be issued. | ||
|
||
PR's that deal with documentation, and other adjacent files (README for example) can be pushed to the _master_ branch. | ||
|
||
When submitting PR's please take into account: | ||
* the project's goals | ||
* PEP8 and the style guide below | ||
|
||
# Testing | ||
- the project's goals | ||
- PEP8 and the style guide below | ||
|
||
## Developer Instructions | ||
|
||
Clone the repository, then install the development dependencies (we recommend using a virtual environment): | ||
|
||
```bash | ||
python -m venv .venv # create virtual environment in .venv | ||
source .venv/bin/activate # activate the virtual environment | ||
pip install --requirement requirements.txt # install development dependencies (includes asammdf in editable mode) | ||
``` | ||
|
||
Now you can start developing. If you are using VSCode, the virtual environment should be detected automatically. If not, open the command panel `Ctrl + Shift + P` and search for `Python: Select Interpreter`. | ||
|
||
## Testing | ||
|
||
You can use tox to run tests locally. Example for the unit tests with Python version 3.7: | ||
You can use tox to run tests locally. Example for the unit tests with Python version 3.9: | ||
|
||
```console | ||
tox -e py27 | ||
tox -e py39 | ||
``` | ||
|
||
Otherwise, you can just push and the tests will be run by GitHub Actions. | ||
|
||
# Style guide | ||
## Style guide | ||
|
||
Just run [*black*](https://black.readthedocs.io) on modified files before sending the PR. There is no need to reinvent the wheel here! | ||
Just run [_black_](https://black.readthedocs.io) on modified files before sending the PR. There is no need to reinvent the wheel here! | ||
|
||
**Tip**: Add a pre-commit hook with `pip install pre-commit && pre-commit install` | ||
**Tip**: install Git hooks using pre-commit `pre-commit install --install-hooks` |
Oops, something went wrong.