mahmoudimus / nose-timer

A timer plugin for nosetests (how much time does every test take?)
MIT License
126 stars 33 forks source link

Colour on Travis CI #46

Closed hugovk closed 9 years ago

hugovk commented 9 years ago

Is it possible to use colour on Travis CI?

This build uses this command with both --timer-ok and --timer-warning flags:

coverage run --append --include=PIL/* -m nose -vx Tests/test_*.py --with-timer --timer-top-n 10 --timer-ok 1s --timer-warning 10s

There are some tests taking up to 46 seconds, but it's all in black and white.

Am I missing something, or is it not possible to use colour with Travis CI?

skudriashev commented 9 years ago

@hugovk, have you tried this locally? Do you have the same behavior?

@mahmoudimus, can we update the package version? It seems to me that v0.3.0 is outdated.

hugovk commented 9 years ago

@skudriashev Yes, it works locally on a Mac: I get normal colour for tests quicker than the --timer-ok value, yellow for tests slower than that, and red for tests slower than --timer-warning.

skudriashev commented 9 years ago

@hugovk, checkout this - https://travis-ci.org/mahmoudimus/nose-timer/jobs/42374563 This is run of last version of nose-timer - looks like everything works fine. I suppose this is because outdated package (v0.3.0) is used.

skudriashev commented 9 years ago

@mahmoudimus, @e0ne, could you, please, release new version with the latest changes?

hugovk commented 9 years ago

@skudriashev Thanks, that's the problem. If I install the latest version from git using pip install git+git://github.com/mahmoudimus/nose-timer.git it's coloured correctly: https://travis-ci.org/hugovk/Pillow/jobs/42378305

@mahmoudimus, @e0ne, +1 on a new release.

Thanks all!

mahmoudimus commented 9 years ago

yep will release today

On Friday, November 28, 2014, Hugo notifications@github.com wrote:

@skudriashev https://github.com/skudriashev Thanks, that's the problem. If I install the latest version from git using pip install git+git:// github.com/mahmoudimus/nose-timer.git it's coloured correctly: https://travis-ci.org/hugovk/Pillow/jobs/42378305

@mahmoudimus https://github.com/mahmoudimus, @e0ne https://github.com/e0ne, +1 on a new release.

Thanks all!

— Reply to this email directly or view it on GitHub https://github.com/mahmoudimus/nose-timer/issues/46#issuecomment-64877942 .

Mahmoud Abdelkader http://mahmoudimus.com/blog

hugovk commented 9 years ago

Thanks, please let me know when it's done.

mahmoudimus commented 9 years ago

@skudriashev @e0ne what do you guys think about having a github hook to auto-publish to pypi on tag?

skudriashev commented 9 years ago

@mahmoudimus, sure, +1:)

hugovk commented 9 years ago

You can have Travis CI deploy to PyPI on tag:

http://docs.travis-ci.com/user/deployment/pypi/

e0ne commented 9 years ago

@mahmoudimus good idea, i like it!

mahmoudimus commented 9 years ago

@hugovk thanks! Implemented here: #49

skudriashev commented 9 years ago

@mahmoudimus, I've just created tag v0.4.0, but I'm not sure anything happened. Do you know why?)

skudriashev commented 9 years ago

@mahmoudimus, Ah, ok, it is still in the queue)

skudriashev commented 9 years ago

@mahmoudimus, look like not everything is fine:

Skipping deployment with the pypi provider because this branch is not permitted to deploy

mahmoudimus commented 9 years ago

@skudriashev hmm what branch is it even talking about? let me check.

mahmoudimus commented 9 years ago

@skudriashev looks like I missed master :-( You fixed it though. Should be on Pypi now.

skudriashev commented 9 years ago

@mahmoudimus, Yes, but doesn't seems like it helped(

skudriashev commented 9 years ago

@mahmoudimus, ah, I need a new tag...

mahmoudimus commented 9 years ago

@skudriashev its still building though on travis: https://travis-ci.org/mahmoudimus/nose-timer

mahmoudimus commented 9 years ago

@skudriashev yeah maybe 0.4.1 now :)

skudriashev commented 9 years ago

@mahmoudimus, I'll do that. Cuz it says now:

Skipping deployment with the pypi provider because this is not a tagged commit

skudriashev commented 9 years ago

@mahmoudimus, done :)

hugovk commented 9 years ago

Thanks both!

(Just a note in case it comes up for others: pip install -U nose-timer worked fine on Windows. On Mac I got a permissions error, but pip uninstall nose-timer and pip install nose-timer worked.)