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

rope.base.ast and rope.base.astutils #582

Merged
merged 8 commits into from Dec 10, 2022
Merged

Commits on Dec 9, 2022

  1. Copy the full SHA
    7038777 View commit details
    Browse the repository at this point in the history
  2. Copy the full SHA
    d2c1ebf View commit details
    Browse the repository at this point in the history
  3. Silence unnecessary warnings

    lieryan committed Dec 9, 2022
    Copy the full SHA
    29907ae View commit details
    Browse the repository at this point in the history
  4. Remove outdated comment that no longer makes sense nor is relevant fo…

    …r current code
    
    Background: rope used to use `compile(..., ast.PyCF_ONLY_AST)` instead
    of `ast.parse()`
    lieryan committed Dec 9, 2022
    Copy the full SHA
    dd8d2f3 View commit details
    Browse the repository at this point in the history
  5. Remove usage of standard library ast

    rope code should not use standard library ast directly
    lieryan committed Dec 9, 2022
    Copy the full SHA
    0cfa540 View commit details
    Browse the repository at this point in the history
  6. Rename rope.base.{astutils->nameanalyze}

    `astutils` is a very bad module name for this, many parts of static
    analysis can be described as "utilities to work with ast", so it's not
    really descriptive enough as to what get_name_levels() and
    `_NodeNameCollector` really does. The content of this current module
    isn't actually generic utilities either, they do something very
    specific, and so calling it `astutils` is also very misleading.
    lieryan committed Dec 9, 2022
    Copy the full SHA
    ff43b71 View commit details
    Browse the repository at this point in the history
  7. Copy the full SHA
    9d30918 View commit details
    Browse the repository at this point in the history

Commits on Dec 10, 2022

  1. Copy the full SHA
    4209cd5 View commit details
    Browse the repository at this point in the history