-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Chore/update readme #115
Chore/update readme #115
Conversation
README.md
Outdated
- Install options: | ||
- ```bash | ||
brew install java | ||
``` | ||
- ```bash | ||
brew install openjdk | ||
``` | ||
- Additional tasks | ||
|
||
- ```bash | ||
java -version | ||
``` | ||
|
||
If you get "Unable to locate a Java Runtime", proceed to next step. | ||
|
||
- Ensure your symlink is correctly mapped (these instructions will also appear after you brew install java) | ||
|
||
- ```bash | ||
sudo ln -sfn /opt/homebrew/opt/openjdk/libexec/openjdk.jdk \ | ||
/Library/Java/JavaVirtualMachines/openjdk.jdk | ||
``` | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All of these instructions are only for homebrew, right? Similar to the other install options sections in the Readme, can you put this under a 'Install options:' bullet point and then a 'Install using homebrew:' bullet point? We should probably provide other install options, such as a link for install instructions/options, so that people know they don't have to use homebrew.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure thing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
README.md
Outdated
@@ -59,6 +81,8 @@ A place to learn and share with developers what makes web work accessible. This | |||
|
|||
> See the [Chromedriver issues](#chromedriver-issues) section if you encounter an error related to Chromedriver. | |||
|
|||
> Make sure your server is running in another terminal before running `npm run test`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
While this instruction is mentioned in the testing section, I agree it's a good idea to include it here too. Can we just modify this slightly to specify it's the 'local' server that needs to be running?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Updates instructions for installing JAVA