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 pre-commit hooks #199

Merged
merged 3 commits into from
Jul 20, 2023
Merged

Add pre-commit hooks #199

merged 3 commits into from
Jul 20, 2023

Conversation

J535D165
Copy link
Owner

  • Make use of precommit
  • Add black and autoupdate

@J535D165 J535D165 merged commit 9e59fc2 into master Jul 20, 2023
20 checks passed
@J535D165 J535D165 deleted the precommit branch July 20, 2023 12:53
EdAbati added a commit to EdAbati/recordlinkage that referenced this pull request Feb 10, 2024
* Fix usage examples (J535D165#190)

* Fix broken links (J535D165#186)

This commit fixes broken links in readme.

* Add threshold None and label docstrings for String (J535D165#189)

* Add support for pandas==2 (J535D165#192)

* Replace setup.py by pyproject.toml (J535D165#195)

* Lint with Ruff and format with Black (J535D165#196)

* Lint with Ruff and format with Black

* Fix more lint issues

* Fix datasets submodule

* Fix all lint errors

* Fix importerror

* Replace flake8 in github action by ruff

* Fix linter

* Fix abstractmethod errors

* Fix test with incorrect error

* Update ci-workflow.yml

* Update CI docs generation and CI pipeline (J535D165#197)

* Bump minimal versions of dependencies

* Update the docs CI pipeline (J535D165#198)

* Add requirements to .readthedocs.yaml

* Bump minimal Python version in documentation

* Add pre-commit hooks (J535D165#199)

* Update CI pipeline for publishing package

* disable docs and publish GH actions

* only trigger on PR

* fixed linting

* updated to latest ruff

* Update GitHub Actions workflows

---------

Co-authored-by: Martinho Hoffman <[email protected]>
Co-authored-by: andyjessen <[email protected]>
Co-authored-by: David GG <[email protected]>
Co-authored-by: Jonathan de Bruin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant