ress/graypy — comment on commit d96aabe9ffa5a503f2e7d7e57e63bb07dbc13ae5
File:  line 0
By severb:

I have a few kind suggestions:
* Please rename `skipList` into `skip_list` as a more "Pythonic" naming convention than CamelCase.
* I suggest adding `id` in `skip_list` based on the note in the `GELF` documentation:
> Libraries SHOULD not allow to send id as additional field (_id), server MUST skip the field because it could ovveride the MongoDB _key field else.
* I think rewriting the "for - if - continue" construct as "for - if not" would be easier to read. Another approach would be to iterate on a set difference (i.e. `for key in set(record.__dict__) - skip_list` where `skip_list` is defined as set. Doing this will also render the if useless).