-
Notifications
You must be signed in to change notification settings - Fork 9
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
Unittests #25
Comments
travis? 👍 |
Looks cool! Maybe we can help to get it running on your machine. What's missing? |
@Nesquick Back to topic: doku say that we can run with --detach or -B task :travis do
system "jekyll serve --detach --port 4000"
end # task :travis |
Could you add a section in README.md how to run the tests? My only concern is what this setup is a bit complex, what do we need grunt for? Maybe you could explain what's happening there. I would like to help you with this, but I don't see the latest travis build nor am I able to get this running on my machine. |
Grunt is the only way I found to call node.js tests directly, if we can call the JS tests directly its better than my currently solution. I search only a few minuets for this solution. Yes I can add this in to readme.md but in front of this I this stuff finished. |
all my builds you found on travis |
@Nesquick https://travis-ci.org/ClemensSahs/phpughh.github.com/builds/15333301 I rebase this and send a PR |
@ClemensSahs @Nesquick awesome! |
@ClemensSahs Thanks a lot! I will check this out en details to see if we can get rid of grunt and simplify the setup. But again, thank you very much for adding this 👍 So is this the current branch? https://github.com/ClemensSahs/phpughh.github.com/compare/feature;travis_2 Could your write one or two sentences what's happening and how to set it up? |
@Nesquick branch: feature/travis I send PR now ;) readme will follow sone as possible |
yep, lets continue this at the PR 😃 |
I guess this will never happen but would be super awesome ❤️
The text was updated successfully, but these errors were encountered: