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

Add ability to read file object #376

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

simonghrt
Copy link

A function read_file_object has been added in order to be able to
directly read a file object (useful for many usecases)

A function read_file_object has been added in order to be able to
directly read a file object (useful for many usecases)
@simonghrt
Copy link
Author

This PR is based on [https://github.com//pull/331](this one) as it's the same use case. I added a test and modified the code in a simpler way (and following PEP8 guidelines). I've read your previous comment about the read_file_obj in the main interface but I didn't see any other alternatives. Let me know if you wish any modifications

@yeus
Copy link

yeus commented Oct 5, 2020

just realizing this is a somewhat old? repository?

@simonghrt if you read this, resend the pull request over here:

@FelixSchwarz
Copy link

It would be really nice to get this functionality in camelot. @simonghrt Can you create a new PR for the camelot-dev repo?

@stevenkissack
Copy link

We'd also love this over using tempfile!

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

Successfully merging this pull request may close these issues.

4 participants