search

opensecuritycontroller / community

Apache License 2.0
15 stars 42 forks source link

Evinwilkins/development #26

Closed emanoelxavier closed 7 years ago

emanoelxavier commented 7 years ago

@evinwilkins due to recent code migrations I am creating a new pull request from your branch. This will show as a new pull request but you can still refer to the pending and existing comments on the previous pull request #24 . @hmthomax evin will be addressing your comments on this PR.

evinwilkins commented 7 years ago

All work completed on my end. Thank you!

evinwilkins commented 7 years ago

All addressed but the question for Emanoel above. Thanks!

emanoelxavier commented 7 years ago

@evinwilkins last comment from me: Replace the paragraph starting with "See Requirements if you choose..." with "See Requirements above or if you choose to clone the repositories differently ensure to set the git hooks under the /hooks/ directory for each repository, for details see the script."

evinwilkins commented 7 years ago

Okay....latest comments have been addressed.

emanoelxavier commented 7 years ago

@evinwilkins The links are not right. "See [Requirements](#requirements) above or if you choose to clone the repositories differently ensure to set the git hooks under the /hooks/ directory for each repository, for details [see the script](./scripts/clone-repos.sh)"

evinwilkins commented 7 years ago

@emanoelxavier Ah, I see now that I need to click Edit. These links are now updated.