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

Save report to file #2

Open
bregman-arie opened this issue Oct 12, 2016 · 9 comments
Open

Save report to file #2

bregman-arie opened this issue Oct 12, 2016 · 9 comments

Comments

@bregman-arie
Copy link
Owner

Right now reqwise will print the report to the console. We should allow to keep the report in a file:

reqwise -o report.txt

@mramanathan
Copy link

@bregman-arie, Is this issue about capturing output from print statements and redirect them to output file as per users' choice ?

If so, existing functionality (which I have not gone thru' fully yet), should be retained as well (i.e. if there are print statement redirecting the output to standard console) ?

@bregman-arie
Copy link
Owner Author

@mramanathan yes, if user specified '-o <output_file>' then the output of the logging lines (there is no use of print statements) should be also redirected to a file in addition to console output.

Thanks for the interest.

@ShanikaEdiriweera
Copy link

Hi @bregman-arie, I would like to tackle this issue with your guidance.

@ShanikaEdiriweera
Copy link

@bregman-arie I have done the fix. How do I run the modified code on local machine to test?

@ghost
Copy link

ghost commented Jan 27, 2017

@bregman-arie can i work on this?

@bregman-arie
Copy link
Owner Author

@ShanikaEdiriweera sorry for the late reply. Missed the notification in mail.
Run with 'reqwise -o report.txt' and see if the file created. If yes, send a pull request and I'll merge it :)

@bregman-arie
Copy link
Owner Author

@shreab373 sure! thank you :)

@ShanikaEdiriweera
Copy link

I still can run reqwise -o report.txt since issue #6 is still there.

@bregman-arie
Copy link
Owner Author

@ShanikaEdiriweera did you run pip install -r requirements.txt from the repo itself?

pypi didn't have latest version. I updated it now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants