Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

Minor README updates #615

Merged
merged 3 commits into from Aug 3, 2016
Merged

Minor README updates #615

merged 3 commits into from Aug 3, 2016

Conversation

unobe
Copy link
Contributor

@unobe unobe commented Aug 3, 2016

Changes proposed in this pull request:

  • Update README to fix missing parenthesis
  • Update README to add note for Mac OS users to run CouchDB

cc @HospitalRun/core-maintainers

@jkleinsc
Copy link
Member

jkleinsc commented Aug 3, 2016

@unobe Thanks for the PR. I am concerned about adding the statement:

(Note: If on Mac, you need to make sure CouchDB can be run: go to System Preferences > Security & Privacy > Allow apps downloaded from: Anywhere)

Because this is a system wide setting, not one particular to CouchDB, it essentially is suggesting to a user that they should be lax about the security of their OS. I would prefer to recommend users just allow CouchDB specifically using instructions like the ones here under the section
How to open an app from a unidentified developer and exempt it from Gatekeeper:
https://support.apple.com/en-us/HT202491

@unobe
Copy link
Contributor Author

unobe commented Aug 3, 2016

@jkleinsc I've modified the patch to just refer to Apple's documentation: let the user understand what they want to do security-wise.

@jkleinsc
Copy link
Member

jkleinsc commented Aug 3, 2016

@unobe Looks good to me. I'll merge it in. Thanks again!

@jkleinsc jkleinsc merged commit e75744f into HospitalRun:master Aug 3, 2016
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.

None yet

2 participants