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

Generalize write_job_output to an arbitrary object instead of "File" #208

Open
crich011 opened this issue Sep 3, 2019 · 2 comments
Open
Milestone

Comments

@crich011
Copy link

crich011 commented Sep 3, 2019

In 2.0.0, we released write_job_output which adds a file to the job outputs. This mechanism can be extended without much effort to include all admissible platform object types.

@patr1ckm
Copy link
Contributor

patr1ckm commented Sep 4, 2019

Yes, we should absolutely do this.

  • I think Table would work transparently.
  • It would be really nice ifJSONValue worked automatically too - the function serializes to JSON, then uploads.

I don't think it makes sense for this function to support Report, Project, or Credential.

@crich011
Copy link
Author

crich011 commented Sep 4, 2019

Makes sense to me!

@patr1ckm patr1ckm added this to the 2.2 milestone Sep 6, 2019
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

2 participants