Skip to content

Setup linting GitHub Action for both Python and Cython #23

Description

@wbkboyer

I noticed that ruff is mentioned (albeit commented-out) in main.yaml for linting purposes; however, ruff does not appear to support linting for Cython (see astral-sh/ruff Issue #10250), so we'd likely need to add an additional linter such as cython-lint (cython-lint on PyPI). Whichever linter is used, we need to decide which requirements.txt to which to add this dependency (see #20 ).

The Python linting should cover the planarity/planarity_functions.py, planarity/planarity_networkx.py, and scripts in the examples/, and whichever Cython linting solution we choose should be directed towards the remaining contents of the planarity/ directory (including subpackages in the future).

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions