-
Notifications
You must be signed in to change notification settings - Fork 137
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
WIP: Feature/coverall integration #195
base: master
Are you sure you want to change the base?
Conversation
Fuco1
commented
Oct 14, 2016
- Remove emacs 23 tests
- Run tests using the evm & cask combo (as is standard these days)
- Run tests for all 24.* versions as well as 25.1
- Add coveralls support
Looking good! |
Can't get undercover to work, it alawys gives me
I don't see anything that should compile dash before tests. Hmm. |
1501d9c
to
728d44d
Compare
So the problem was with doing |
Might I suggest separating out the EVM changes from moving to coveralls? AFAICS the EVM + cask work is ready to merge. |
That's a good idea. I'm finishing next smartparens release now so I'll probably spend the weekend on that. Do you want to try and get this separated? |
b903a0f
to
2e8d667
Compare
Is this still something we want to have? |