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.

Sign up for GitHub

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

Jump to bottom

Lint checking: Allow user configurable ignoring of flakes #22

Closed
ceball opened this issue Jun 14, 2019 · 0 comments · Fixed by #19
Closed

Lint checking: Allow user configurable ignoring of flakes #22

ceball opened this issue Jun 14, 2019 · 0 comments · Fixed by #19

Comments

@ceball
Copy link
Contributor

ceball commented Jun 14, 2019

Maybe a regex based ignore list for flexibility? E.g. to allow deliberate hvplot imports with side effects, you'd add this to the list: "'hvplot\..*' imported but unused".

Or should it be more specific (e.g. support for ignoring unused names)? This would probably be more effort to implement, though I think there aren't too many categories of things pyflakes would ever complain about.
@ceball ceball changed the title Allow user configurable ignoring of flakes Lint checking: Allow user configurable ignoring of flakes Jun 14, 2019
@ceball ceball mentioned this issue Jun 14, 2019
Merged
4 tasks
ceball added a commit that referenced this issue Sep 14, 2019
@ceball
Initial go at next version, improve maintainability (#19)
7202ac3 
Adds some basic missing features, and cleans up some internals.

Note: some ipython magic handling has changed (should be an improvement/more consistent behavior).

New features:

  * support for flake messages to ignore (by regex) - fixes #22
  
  * support for magics blacklist (i.e. magics that if present cause a flake) - fixes #21

  * added support for script & capture cell magics (mainly as a demo) - fixes #24

  * increased the number of builtin magics that will be silently ignored (as they don't interact with the python process), e.g. bookmark, edit, who, etc - fixes #24

  * lint failures can be set to be warnings only (potentially useful for systems that report warnings)

Internal changes:

  * Separated out parts into modules:
  
      * the original/basic functionality (checking notebooks run without errors)
  
      * "verifying" (copied from datashader, which was copied from bokeh?)
      
      * lint/flake checking (using pyflakes)
  
      * magics handling for flakes
  
  * Improve lint/magics debugging by storing various intermediate files.
  
  * Simplify lint checking of magics, making behavior more consistent.
  
  * Replaced custom/hacky magics parsing with stuff from ipython itself (fixes #8)

  * MAGICS_TO_IGNORE and SIMPLE_MAGICS: separated cell and line magics (internal change to support more magics) - fixes #23 

  * Improved unit test coverage (think all code is now covered except "verify").
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Initial go at next version, improve maintainability holoviz-dev/nbsmoke
1 participant
@ceball