Skip to content

Setup linting GitHub Action for both Python and Cython #23

@wbkboyer

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
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions