-
Notifications
You must be signed in to change notification settings - Fork 203
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
Intern 3.4.0 and instructions in CONTRIBUTING.md #1343
Comments
This follows the #982 comment updating to Intern 3.4.0 |
Probably @andreastt or @miketaylr have an idea of the missing step |
My Firefox Nightly is 54. The Firefox production release is 51.
OK Let's try with the most recent one. export JAVA_HOME="/Library/Internet Plug-Ins/JavaAppletPlugin.plugin/Contents/Home" Then
WORKING! So be careful, basically you need a recent version of java. Probably should be tested in a command. |
Are you giving it a user and pw to run the auth tests with? edit: https://github.com/webcompat/webcompat.com/blob/master/.travis.yml#L61 |
I guess we need to say whatever the minimum Java requirement is here: https://github.com/webcompat/webcompat.com/blob/master/CONTRIBUTING.md#installing-java |
Ah, this is documented in a comment, but not contributing.md, i'll write a quick docs patch: https://github.com/webcompat/webcompat.com/blob/master/.travis.yml#L1 |
@karlcow also, i would recommend you delete your Java Applet Plugin (unless you rely on that for some old banking websites or something), and upgrade |
yes :)
I will give it a try. I remember upgrading with downloading and double clicking. |
(that said, do whatever works for you. computers are scary. 👻) |
Currently if you follow strictly what is in CONTRIBUTING.md, the tests do not run.
https://github.com/webcompat/webcompat.com/blob/master/CONTRIBUTING.md#functional-tests
In one shell
then opening a new bash window:
The text was updated successfully, but these errors were encountered: