Skip to content

[TECH DEBT]: Skip parsing notebook if present in the NotebookLinter._python_tree_cache #3552

@JCZuurmond

Description

@JCZuurmond

Is there an existing issue for this?

  • I have searched the existing issues

Problem statement

Skip parsing notebook if present in the NotebookLinter._python_tree_cache. This means the notebook was already parsed because it was referred by another notebook using a RunCell

Proposed Solution

Use the cache to get the Python tree instead of parsing.

Notes:

  • this requires to extend the cache to contain failures too.
  • A notebook can be run from multiple parent notebooks. We need to handle that situation

Additional Context

No response

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or requestinternalthis pull request won't appear in release notesmigrate/codeAbstract Syntax Trees and other dark magicneeds-triage

    Projects

    Status

    Todo

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions