This repository has been archived by the owner on Jun 11, 2021. It is now read-only.

Empty dict as default value #1

Open

graup opened this issue Nov 7, 2017 · 0 comments

graup commented Nov 7, 2017 •

edited

Loading

Just spotted this and thought I'd let you know

This is a potential bug: https://github.com/kunanit/mooclet-engine/blob/069c8aaca2d82cd0841ab1ece5b4eadefbedbd29/mooclet_engine/engine/models.py#L28

See: https://stackoverflow.com/a/26320938/700283

/edit:

I guess with the current code it's not really a problem, but it's still a "code smell" I think.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.