Jakub Žádník
Jakub Žádník
Can you please paste the file contents into the issue? It's easier than downloading and unzipping the archive. If you define a command that overrides a builtin, save the builtin...
> I don't see how but sure So that anyone reading through the issue can immediately see what the problem is without downloading any files. This is definitely a parser...
It also doesn't look like a cyclic import. We have a parser guard against cyclic import, and this one doesn't cyclically import anything. But I might actually have a clue...
Let's not close it until the replacement is complete. AFAIK we don't have umkdir PR up yet, so fixing this would still help. The `expect()` in the original code should...
(Nevermind, we already have `umkdir` merged in, so we might as well remove the old `mkdir` soon.)
I believe this is due to `source` parsing missing protection against circular sourcing. We have this protection in modules where you can't do circular imports, but it's most likely missing...
I think this might be because we store parsed files into a cache in the parser to avoid reparsing them, but we don't clear the cache when we're done. The...
I'd vote for yes, until we have a better fix.
The above comment is meant globally, i.e., wrap entire Nushell into `catch_unwind()`.
Thanks for pointing it out. We should be able to just remove the dependency, it's only used in plugins example tests.