fedora-infra/fedmsg

It would be nice if ``fedmsg.publish`` documented the type for ``msg``

Closed this issue · 1 comments

We're in a situation at the moment where we have a JSON bytestring encoded with UTF-8 and it would be nice to hand this directly to fedmsg.publish, but its API doesn't appear to state what it'll accept. I don't see anything in the code that wouldn't work with this (although I just glanced through the function), but I'm hesitant to use the API this way if it's not actually a documented part of the API.

It would be good to document what's acceptable and make sure the function lives up to that documentation.

Fixed in #408