Skip to content

Commit e27d827

Browse files
committed
clarify
1 parent 69631d9 commit e27d827

File tree

2 files changed

+8
-6
lines changed

2 files changed

+8
-6
lines changed

CONTRIBUTING.md

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,18 @@
11
# CONTRIBUTING
22

3-
> note: this codebase is currently (and historically) rather entangled with [osf.io](https://osf.io), which has its shtrove at https://share.osf.io/trove -- stay tuned for more-reusable open-source libraries and tools that should be more accessible to community contribution
3+
> note: this codebase is currently (and historically) rather entangled with [osf.io](https://osf.io), which has its shtrove at https://share.osf.io -- stay tuned for more-reusable open-source libraries and tools that should be more accessible to community contribution
44
55
For now, if you're interested in contributing to SHARE/trove, feel free to
66
[open an issue on github](https://github.com/CenterForOpenScience/SHARE/issues)
77
and start a conversation.
88

9-
## Requirements
9+
## Required checks
1010

11-
All new changes must pass the following checks with no errors:
12-
- unit tests: `python -m pytest -x tests/`
11+
All changes must pass the following checks with no errors:
1312
- linting: `python -m flake8`
1413
- static type-checking (on `trove/` code only, for now): `python -m mypy trove`
14+
- tests: `python -m pytest -x tests/`
15+
- note: some tests require other services running -- if [using the provided docker-compose.yml](./how-to/run-locally.md), recommend running in the background (upping worker ups all: `docker compose up -d worker`) and executing tests from within one of the python containers (`indexer`, `worker`, or `web`):
16+
`docker compose exec indexer python -m pytest -x tests/`
1517

16-
All new changes should also avoid decreasing test coverage, when reasonably possible.
18+
All new changes should also avoid decreasing test coverage, when reasonably possible (currently checked on github pull requests).

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
77
SHARE/trove (aka SHARtrove, shtrove) is is a service meant to store (meta)data you wish to keep and offer openly.
88

9-
note: this codebase is currently rather entangled with [osf.io](https://osf.io), which has its shtrove at https://share.osf.io/trove -- stay tuned for more-reusable open-source libraries and tools for working with (meta)data
9+
note: this codebase is currently (and historically) rather entangled with [osf.io](https://osf.io), which has its shtrove at https://share.osf.io -- stay tuned for more-reusable open-source libraries and tools for working with (meta)data
1010

1111
see [ARCHITECTURE.md](./ARCHITECTURE.md) for help navigating this codebase
1212

0 commit comments

Comments
 (0)