Details
Diff Detail
- Repository
- rCOMM Comm
- Branch
- master
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
| docs/dev_environment.md | ||
|---|---|---|
| 613–618 | Opted to go with this at the top of each of the successive sections that require the keyserver to be running. If it's too wordy, we can provide a link to the "Running keyserver" section or another alternative. But this information helps someone who just looks at/searches for the "Running web app" section without reading any other parts of the document. | |
| 627 | This was pushed down from the "Running keyserver" section because it makes more sense here. The web app cannot be viewed in the developer's web browser until:
Thus, by moving this statement below the code to run the web app, it now more accurately represents what's running. | |
| 629 | This was added to match the similar paragraphs in the below sections. The paragraphs in the "Running web app" and "Running mobile app" sections both say "This command..." | |
| 647 | This was pushed down from the "Running keyserver" section because it makes more sense here. The landing page cannot be viewed in the developer's web browser until:
Thus, by moving this statement below the code to run the landing page, it now more accurately represents what's running. | |
| 689 | This was something I must have missed in D4257, I'm not sure how. Fixed it here because it's a really quick change. | |
Thank you for moving your detailed notes from the previous diff!! Makes it way easier to review :)