-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
proposal: add Deno.ConfigFile
#24628
Comments
I don't think this should be built-in because if breaking changes happen to the config (ex. like lint.files being removed) then the type will change based on what version of Deno the code is run with rather than the version of the deno config that the code supports. |
Isn't that true for any type in the |
I don't think so because those are runtime APIs. |
I think it still might provide more value than not having it. @marvinhagemeister, any thoughts? |
I will reiterate my support here, though I understand and sympathize with David's hesitation. As a framework author, it really sucks just having to "guess" where the |
Maybe a new env variable that says the path to the config file that was used? Then when a module needs it, they can read the files content themselves, instead of requiring the user of the module to provide the path/info. |
This is about having an interface that represents the |
Ah sorry, right. I guess it was confusing because you closed #19322 "in favor" of this one when they are not doing the same thing. I misread the initial issue description. |
Sometimes projects have to read a
deno.json
file (e.g. in Fresh here). In such cases, having a type that makes working with the file easier, from theDeno.*
namespace, could be quite helpful, and prevent users from having to come up with their own types (another example in Fresh here).The text was updated successfully, but these errors were encountered: