Results 55 comments of Thom Wright

Hey, thanks for the PR. Could you look at #85 and tell me if you think this would be better written as `cause: Error` instead of `reason: string`? Thanks

I think the mitigation here is generally to make your migrations backwards-compatible with the previous version of your application. Even if you only run one migration at a time, you...

Hi, thanks for the bug report. Could you explain what practical problem this causes? Do you have apps running on different OSs connecting to the same shared database and both...

Great, thanks. That's useful context. Since this is essentially a breaking change on Windows, in my mind this potentially needs a new major version. I have some other changes I'd...

Yeah I think that sounds like a pretty sensible idea. Sorry I didn't reply sooner, I don't spend much time on this library anymore. Happy to take a PR for...

No. This approach was suggested in the thread I linked to, but was thought to be a breaking change: > However it could break things for users who have their...

I also didn't know that! This is why I linked to #33 - it's in there. See here for some documentation: https://www.postgresql.org/docs/9.6/ddl-schemas.html#DDL-SCHEMAS-PATH

Thank you for the issue. Looking at how `cause` is used in the code, it seems like it should probably be an `Error` anyway! Technically this will be a backwards-incompatible...

Interesting case! Thanks for the issue. I must admit, my time for maintenance on this library is minimal, so it's unlikely I'll spend time implementing this. That said, this is...