Colin Snover

Results 72 comments of Colin Snover

Here is another error that came up at some point which I assume is related to this PR: ``` File navigation : string indices must be integers Traceback (most recent...

Thanks, I’ll reapply in the next while and let you know how it works here.

So far so good! I’ll continue banging on this the next couple days and let you know if I run into anything. Thanks for the hard work!

I don’t know if you plan on addressing it, but the one issue that still exists for me (that existed and was worse before this patch) is where the imported...

One way that seems to be possible to consistently reproduce the issue is: 1. Ensure `hot_exit` is enabled in Sublime (it is by default) 2. Open a TypeScript file that...

> Because typescript allows for circular imports, there is no logical way to determine the correct root file. I believe the “root” file would be the file that is open...

@Phaiax Thanks for your thoughtful response. I’m planning to read through it in greater detail and respond to any points when I have a free moment.

Well, in an effort to be more helpful, I did a little digging and see that (1) this issue should probably be in the graphile-build repository (sorry about that), and...

> Removing a non-null constraint is a breaking change, so this certainly wouldn’t be done by default to avoid breakage if you implemented RLS later. I can understand not wanting...

If we are going to include something to do this, I’d prefer it to not be something that says “you are probably going to want something more robust” in its...