Skip to content

Commit

Permalink
A few small doc tweaks & grammar fixes (#616)
Browse files Browse the repository at this point in the history
* A few small documentation tweaks to fix grammar mistakes and make things more readable

* Additional edit suggested in PR

Co-authored-by: Ian Varley <[email protected]>
  • Loading branch information
ivarley and Ian Varley authored Apr 21, 2020
1 parent 028a3c0 commit 6852c9d
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 15 deletions.
18 changes: 6 additions & 12 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -127,7 +127,7 @@ git rebase develop
git push -f
```

- Documentation on [create a Pull Request (PR) on Github](https://help.github.com/articles/using-pull-requests/) for review and merging.
- Documentation on how to [create a Pull Request (PR) on Github](https://help.github.com/articles/using-pull-requests/) for review and merging.

**Note**: Even if you have write access, do not work directly on `master` or push directly to `develop`! All work is done against `develop` reviewed and merged via PRs, and ultimately `develop` gets merged into `master` for tagged code releases.

Expand All @@ -143,13 +143,13 @@ This project utilizes the [Jest](https://jestjs.io/) testing framework to test t

### Adding new components/tests

When adding new components to the project, make sure to add at least snapshot test. The test files are located in the `__tests__` directory at the same level as the component you are testing. For instance, if you are adding a new view called `TestView` under the `views` directory, you would add a `TestView.spec.js` file in the `views/__tests__` directory.
When adding new components to the project, make sure to add at least a snapshot test. The test files are located in the `__tests__` directory at the same level as the component you are testing. For instance, if you are adding a new view called `TestView` under the `views` directory, you would add a `TestView.spec.js` file in the `views/__tests__` directory.

After making your changes and adding a new test file, make sure to run `yarn test` to make sure you don't have any test failures.
After making your changes and adding a new test file, run `yarn test` to make sure you don't have any test failures.

### Snapshot failures

The most common failure you will most likely run into is a snapshot failure. [Snapshot testing](https://jestjs.io/docs/en/snapshot-testing) ensures that a component renders the same output after it has been modified. If you receive a snapshot failure, check the differences in the snapshot failure to make sure they are expected changes. If they are all expected changes then you just need to update the snapshot by running `yarn test -u`. This will update the snapshot file so you can commit the changes.
The most common failure you will run into is a snapshot failure. [Snapshot testing](https://jestjs.io/docs/en/snapshot-testing) ensures that a component renders the same output after it has been modified. If you receive a snapshot failure, check the differences in the snapshot failure to make sure they are expected changes. If they are all expected changes then you just need to update the snapshot by running `yarn test -u`. This will update the snapshot file so you can commit the changes.

### Mocking modules

Expand All @@ -167,15 +167,9 @@ When writing tests for components that use installed packages, we usually don't

_Advanced users may install the `hub` gem and use the [`hub pull-request` command](https://github.com/defunkt/hub#git-pull-request)._

- If not done in commit messages (which you really should do), reference and
- If not done in commit messages (which you really should do), reference and update your issue with the code changes. But _please do not close the issue yourself_.

update your issue with the code changes. But \_please do not close the issue

yourself\_.

- A team member will review the pull request, request change or approve and

merge into the `develop` branch.
- A team member will review the pull request, request change or approve and merge into the `develop` branch.

## Reviewing Pull Requests

Expand Down
6 changes: 3 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,15 +8,15 @@ We’re building the next generation of secure location logging to preserve priv

Location logs provide time-stamped records of where users have been, allowing them to share information with health officials accurately and quickly. This helps support contact tracing efforts to slow the spread of the virus.

What’s truly special about Safe Paths, though, is its privacy protection. Data never leaves user's device without their password entry and explicit consent. The location log generated by Safe Paths cannot be accessed from outside the user’s device, meaning data transfer occurs only if the user chooses to share it with the researcher.
What’s truly special about Safe Paths, though, is its privacy protection. Data never leaves a user's device without their password entry and explicit consent. The location log generated by Safe Paths cannot be accessed from outside the user’s device, meaning data transfer occurs only if the user chooses to share it with a researcher or health official.

----

<img align="right" src="./assets/PreviewUI.png" data-canonical-src="./assets/PreviewUI.png"/>

Safe Paths is a ‘privacy-first’ app that allows you to log your GPS trails on your own phone. The information is stored locally and never shared with anyone (not even with us or MIT) until you explicitly decide to manually export the data. The location log generated by Safe Paths cannot be accessed from outside the user’s device. Location information can be imported and exported by the user and used in other projects and applications.

Safe Paths logs your device’s location once every five minutes and stores 28 days of data in under 100KB of space – less space than a single picture. But what is truly exciting about Safe Paths is its privacy protection.
Safe Paths logs your device’s location once every five minutes and stores 28 days of data in under 100KB of space.

**Home page:** https://covidsafepaths.org and https://safepaths.mit.edu

Expand All @@ -32,7 +32,7 @@ This is a React Native app version 61.5

## Architecture

Please refer to `docs/Private Kit Diagram.png` for a basic overview on the sequencing of generalized events and services that are utilized by Safe Paths.
Please refer to `docs/Private Kit Diagram.png` for a basic overview on the sequencing of generalized events and services that are used by Safe Paths.

## Developer Setup

Expand Down

0 comments on commit 6852c9d

Please sign in to comment.