Patrick Cloke

Results 263 comments of Patrick Cloke

Looks like this needs `black` run against it for formatting! 👍

While testing this I ended up with the `tests` directory installed into `site-packages`, which is generally unwanted. This seems to match what Django has in their `MANIFEST.in`, but I don't...

Interesting. I'll be curious to see what you find! 👍

@djg321 This should work OK with Django migrations (not South), I think. I've certainly done this with `ForeignKey` fields to the `auth.User` model. One thing I _will_ note, however, is...

I've started doing this. I created a bunch of `Resource` classes that have sane APIs on them, e.g. my code now looks like: ```python from YHandler import YahooFantasySports handler =...

@BrutalSimplicity I had forgotten to push my changes after forking. Oops. They're there now. [See here for a diff](https://github.com/mleveck/YHandler/compare/master...clokep:master). Some of the files were kind of gratuitously renamed to be...

@suhassumukhv You're running into #46, essentially the content of a list isn't part of the same tag. This was a design decision (according to that issue).

> > The return value should be IHostResolution, not IResolutionReceiver. > > Why? Just asking. Because the type hint was incorrect when added. The implementation of `GAIResolver`, which was the...

There's also cases of servers improperly implementing rfc1459 vs. strict-rfc1459 (see inspircd/inspircd#1017). Ideally, wouldn't we want this to match how it is done for channel names?