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

Add pytest-testmon pre-commit hook #88

Closed
ns-rse opened this issue Dec 11, 2024 · 0 comments · Fixed by #89
Closed

Add pytest-testmon pre-commit hook #88

ns-rse opened this issue Dec 11, 2024 · 0 comments · Fixed by #89
Assignees
Labels
ci Continuous Integration testing Problems with testing

Comments

@ns-rse
Copy link
Collaborator

ns-rse commented Dec 11, 2024

We now use pytest-testmon as a pre-commit hook on
AFM-SPM/TopoStats as it helps us remember to make sure we don't break any
existing tests when working on the code base.

It would therefore be useful to have it enabled in AFMReader.

pyproject.toml

[project.optional-dependencies]
dev = [
  ...
  pytest-testmon,
  ...
]

.pre-commit-config.yaml

  - repo: local
    hooks:
      - id: pytest
        name: Pytest (testmon)
        entry: pytest --testmon
        language: system
        files: \.py$
@ns-rse ns-rse added testing Problems with testing ci Continuous Integration labels Dec 11, 2024
@ns-rse ns-rse self-assigned this Dec 11, 2024
ns-rse added a commit that referenced this issue Dec 11, 2024
Closes #88

Also adds a few more things to `.gitignore`.
@ns-rse ns-rse closed this as completed in 4c4113b Dec 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Continuous Integration testing Problems with testing
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant