facebookincubator/OnlineSchemaChange

'core' is a pretty presumptious name

Opened this issue · 2 comments

On any python environment in which this installed, for which the instructions recommend the system python, import core will import the core of osc. This could clobber web apps which sometimes use a pattern where there's a core module with base classes for the app. I think the module should be renamed osc to match the package name in setup.py.

Also as part of this it could use entry_points in setup rather than scripts

Thanks @adamchainz for the advice! We will re-organize the directory to address this issue.