Frank Dana

Results 954 comments of Frank Dana

@EverardB There's discussion in PR #1574 about possible approaches to that. Personally, I don't mind the extra length (vertical space is free, the terminal scrolls), I just want each line...

> Doubling the timeout so that we don't accidentally catch the property in the wrong state seems to solve the issue, though if it comes up again we may need...

There are several _hundred_ eslint errors coming up with the code in this PR. @Scroker if you go to the root of your worktree and run `npm install` followed by...

> @Scroker if you go to the root of your worktree and run `npm install` followed by `npm install eslint`, you can use `npx eslint --fix` to correct probably most...

(Note, also, that there are a number of docstring issues that I suspect `--fix` may fix **wrong**, because they're actually being misinterpreted by it due to wrong tags being used...

It is a duplicate. And while I could've cheated my way around our approval requirements by approving and merging _this_ PR instead, I decided to just force through my own...

This PR currently also adds a bunch of debugging to the SFTP plugin, which I'm personally inclined to leave in there permanently. (Except for the one bit that dumps out...

@mavit Yup, sounds the same. (Well, I haven't tried using keyboard nav to open the submenu, but even if that works it's not really useful so I'll just assume it...

OK, I think I have a plan for this, but it's going to mean a fairly substantial re-re-working of the whole setup. I got to thinking: The thing about the...

cc: @nebulosa2007, @JingMatrix