json,mongo

[chayim]

No description provided.

[lgtm-com]

This pull request introduces 7 alerts when merging d951d84 into 1377965 - view on LGTM.com

new alerts:

• 2 for Unused import
• 1 for Testing equality to None
• 1 for Module is imported more than once
• 1 for Unused local variable
• 1 for Missing call to `__init__` during object initialization
• 1 for 'import *' may pollute namespace

[MeirShpilraien]

Looks good 👍
Some small comments, the most important are the events you need to register on.

[lgtm-com]

This pull request introduces 7 alerts when merging e2de52b into 1377965 - view on LGTM.com

new alerts:

• 2 for Unused import
• 1 for Testing equality to None
• 1 for Module is imported more than once
• 1 for Unused local variable
• 1 for Missing call to `__init__` during object initialization
• 1 for 'import *' may pollute namespace

[lgtm-com]

This pull request introduces 7 alerts when merging 8082131 into 1377965 - view on LGTM.com

new alerts:

• 2 for Unused import
• 1 for Testing equality to None
• 1 for Module is imported more than once
• 1 for Unused local variable
• 1 for Missing call to `__init__` during object initialization
• 1 for 'import *' may pollute namespace

[lgtm-com]

This pull request introduces 7 alerts when merging c62a24c into 1377965 - view on LGTM.com

new alerts:

• 2 for Unused import
• 1 for Testing equality to None
• 1 for Module is imported more than once
• 1 for Unused local variable
• 1 for Missing call to `__init__` during object initialization
• 1 for 'import *' may pollute namespace

[lgtm-com]

This pull request introduces 7 alerts when merging ee7b0d0 into 1377965 - view on LGTM.com

new alerts:

• 2 for Unused import
• 1 for Testing equality to None
• 1 for Module is imported more than once
• 1 for Unused local variable
• 1 for Missing call to `__init__` during object initialization
• 1 for 'import *' may pollute namespace

[MeirShpilraien]

Only reviewed the code parts (without the build system changes), maybe @rafie will want to go over the build system.