Skip to content
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

Support output to a file to support tox on CI servers #1070

Closed
johnthagen opened this issue Aug 12, 2016 · 4 comments · Fixed by #4339
Closed

Support output to a file to support tox on CI servers #1070

johnthagen opened this issue Aug 12, 2016 · 4 comments · Fixed by #4339
Labels
Enhancement ✨ Improvement to a component Good first issue Friendly and approachable by new contributors Help wanted 🙏 Outside help would be appreciated, good for new contributors

Comments

@johnthagen
Copy link

Currently tox cannot redirect commands to files, which means when running Pylint from within tox there seems to be no way to output the results to a single (for example HTML) file that a CI server can store for display.

--file-output=y used to be close to what is needed, but it was removed in 2.0, and also created a different file for each module which is difficult to navigate.

Here is the StackOverflow question that relates to this.

flake8 added this feature for the same reason: https://gitlab.com/pycqa/flake8/issues/15

...however my main usage of flake8 is from within tox running under jenkins (using the violations plugin). Since tox doesn't support shell redirection...

@PCManticore
Copy link
Contributor

Why not redirecting pylint command in your tox file into a file?

@johnthagen
Copy link
Author

@PCManticore Because (and this was new to me as I hadn't tried before) tox does not support file redirection, so there is no way to do it that I am aware of.

@acerv
Copy link

acerv commented Mar 27, 2019

Hello,

I've the same problem when running pylint inside tox. And also this is more complicated once you need to develop a multiplatform application.

The only way to save the pylint stdout inside a file, is to tell tox to use bash or cmd, according with the platform, with '>' operator. Unfortunately, there's not a way to do so but creating multiple tox environments once for each platform, that is (of course) a huge waste of space and time.

Perhaps makes the automation system really complicated and difficult to handle.

@PCManticore
Copy link
Contributor

Thanks for the input @acerv It seems this will be definitely useful, so I'd be happy to reintegrate a --file-output functionality for the entire output.

@PCManticore PCManticore added the Good first issue Friendly and approachable by new contributors label Apr 23, 2019
@Pierre-Sassoulas Pierre-Sassoulas added the Help wanted 🙏 Outside help would be appreciated, good for new contributors label Mar 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement ✨ Improvement to a component Good first issue Friendly and approachable by new contributors Help wanted 🙏 Outside help would be appreciated, good for new contributors
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants