-
-
Notifications
You must be signed in to change notification settings - Fork 105
Temporary fix to run tests on Travis #106
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
Conversation
Slamdunk
commented
May 31, 2018
- https://twitter.com/slamzoe/status/1002090034382635008
- https://twitter.com/seldaek/status/1002097015831121920
For Scrutinizer something similar should be adopted: It could also be set with a commit configuration file, if you prefer. |
I think this would be better: I don't have control on the Scrutinizer account at the moment Thanks a lot for the help! |
Done. Scrutinizer triggers seem stalled, but the build passed the stage this PR aims to. |
Hm maybe it needs the complete config. Prior to your changes it was:
|
I've messed up and can't figure out why the variable is not taken into account anymore 😃 |
maybe we can just export it manually in the |
Mmm 😕 |
Ok, the first version of the configuration was good, PR ready to merge |
Thanks very much! |