-
Notifications
You must be signed in to change notification settings - Fork 31
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
Set up ruff
& black
with pre-commit
and document usage
#64
Conversation
# import matplotlib as mpl | ||
# from matplotlib.backends.backend_qt4agg import FigureCanvasQTAgg as FigureCanvas | ||
# from matplotlib.backends.qt_compat import QtCore, QtGui | ||
# | ||
# def test_editor_add_point(): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pulled all the imports associated with this commented block into the commented block. Any reason to keep this function though?
9b8bb41
to
9fbf269
Compare
ruff
& black
with pre-commit
and document usage
f.read(), | ||
os.path.basename(args.colormap), # noqa: F821 | ||
"exec", | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The FIXME
above and noqa
comments document a bug found by ruff
. Will get back to this when I'm ready to dig into the python loading behavior and build a unit test :)
2321819
to
cf11834
Compare
With the pre-commit instructions, should we mention how to install it? |
I think since we link to the docs, people should be able to figure it out. But I don't feel strongly, so happy to add that in if you'd like! |
72f4389
to
8438f66
Compare
8438f66
to
ccf1101
Compare
No description provided.