Skip to content

Commit

Permalink
Merge pull request #7 from Learning-and-Intelligent-Systems/yang
Browse files Browse the repository at this point in the history
README
  • Loading branch information
tomsilver authored Mar 17, 2023
2 parents 49023e6 + 7d5f2f4 commit 444149d
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,10 +5,10 @@ A number of helpful utilities for doing cool things with the Boston Dynamics Spo
(Note: Performing the installation in a separate virtualenv or conda environment is
recommended!)
Users should just be able to run `pip install -e .`
Developers should run `pip install -e .[develop]`
Developers should run `pip install -e ".[develop]"`

## Contributing
- Run `pip install -e .[develop]` to install all dependencies for development.
- Run `pip install -e ".[develop]"` to install all dependencies for development.
- You can't push directly to master. Make a new branch in this repository (don't use a fork, since that will not properly trigger the checks when you make a PR). When your code is ready for review, make a PR and request reviews from the appropriate people.
- To merge a PR, you need at least one approval, and you have to pass the 2 checks defined in `.github/workflows/ci.yml`, which you can run automatically with `./run_ci_checks.sh`, or individually as follows:
- `mypy .`
Expand Down

0 comments on commit 444149d

Please sign in to comment.