-
Notifications
You must be signed in to change notification settings - Fork 557
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
✨ API: add File()
or NewFile()
to API to create a new file
#7388
Comments
File
or NewFile
to API to create a new fileFile()
or NewFile()
to API to create a new file
We already have a However, we have:
So having type Query {
file(name: String!, contents: String!): File!
} The problem is We could wait to add this until a release after v0.12 (i.e., release without current |
On the other hand, One could argue though, that with the addition of |
It's more inconsistent with
I thought about that but I'm not sure. If you don't want to repeat the file name: source.WithFile("", dag.File("file.json", "{}")) But internally that'll be equivalent to: source.WithFile("", dag.Directory().WithNewFile("file.json", "{}").File("file.json")) Which produces a Curious if the merge has more benefits and if deprecating |
What are you trying to do?
Today, there is no way to ask the API for a new file directly. You have to go through
Directory
.We got close to seeing this in the past, but it hasn't happened yet.
Why is this important to you?
Many users have requested this including @sagikazarmark @vito @helderco @jpadams
How are you currently working around this?
The text was updated successfully, but these errors were encountered: