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

MAINT: add cython-lint to test suite #198

Open
fkuehlein opened this issue Sep 28, 2023 · 0 comments
Open

MAINT: add cython-lint to test suite #198

fkuehlein opened this issue Sep 28, 2023 · 0 comments
Assignees
Labels
maintenance something should be improved or is outdated testing

Comments

@fkuehlein
Copy link
Collaborator

As discussed in #195, it would be helpful to extend the style and lint checking to the Cython modules. This comment kindly provides a TODO list already:

You could add cython-lint to our test suite by:

  • adding it to the testing dependencies in setup.cfg,
  • adding a new tox test environment in setup.cfg which calls cython-lint,
  • and configuring cython-lint itself in pyproject.toml.

Note that this will inevitably lead to lots of flags and errors to be resolved.

See the cython-lint repo for reference.

@fkuehlein fkuehlein self-assigned this Sep 28, 2023
@fkuehlein fkuehlein added the maintenance something should be improved or is outdated label Sep 28, 2023
@fkuehlein fkuehlein added this to the New Release milestone Oct 12, 2023
@ntfrgl ntfrgl added the testing label Dec 23, 2023
@fkuehlein fkuehlein removed this from the Release 0.7 milestone Jan 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance something should be improved or is outdated testing
Projects
None yet
Development

No branches or pull requests

2 participants