Skip to content
This repository has been archived by the owner on Jun 19, 2020. It is now read-only.

Update README.md #9

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update README.md #9

wants to merge 2 commits into from

Conversation

jbixon13
Copy link
Member

Related PRs

List related PRs against other branches:

branch PR
other_pr_description #

Related Issues

List related Issues:

link
#3

Todos

  • Tests
  • Documentation

Deploy Notes

Notes regarding deployment the contained body of work.

Steps to Test or Reproduce

Outline the steps to test or reproduce the PR here.

commands to test PR

Impacted Areas in Application

List general components of the application that this PR will affect:

@jbixon13 jbixon13 requested a review from c-malecki March 19, 2020 00:36
@jbixon13 jbixon13 self-assigned this Mar 19, 2020
imalsogreg
imalsogreg previously approved these changes Mar 19, 2020
Copy link

@imalsogreg imalsogreg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, solves the problem. Might (might) be worth a mention how to install nodejs (via a link to nodejs.org) for the uninitiated.

@jbixon13
Copy link
Member Author

@imalsogreg ready for re-review, won't let me request one for some reason

Copy link

@imalsogreg imalsogreg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Continues to LGTM

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants