Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

operator.py module name conflicts with a standard library module's #174

Closed
spencerkclark opened this issue Apr 23, 2017 · 1 comment
Closed

Comments

@spencerkclark
Copy link
Collaborator

spencerkclark commented Apr 23, 2017

This is a somewhat obscure issue, but it came up today as I was updating my emacs configuration on the GFDL systems -- I now use flycheck for doing syntax checking/linting of python, which depends on the standard library's operator module. When I edit files in the aospy repository, python tries to import aospy's version of the operator module, which causes problems for flycheck.

There are two simple solutions here:

  1. Rename aospy/operator.py something else
  2. Since we don't currently use any of the code in aospy/operator.py, we could remove it for now

@spencerahill what are your thoughts? Should we try to fix this?

@spencerahill
Copy link
Owner

I'm in favor of removing it. It was hacky to begin with, and I don't see any use for it on the horizon. PR welcome :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants