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 update-pyproject pre-commit hook #128

Merged
merged 8 commits into from
May 15, 2023
Merged

Conversation

CasperWA
Copy link
Collaborator

Closes #24

This adds a new pre-commit hook with id update-pyproject, which will run ci-cd update-deps to update the required and optional dependencies in a pyproject.toml file using pip index versions.

The hook runs `ci-cd update-deps` to update dependencies in a
`pyproject.toml` file.

A new documentation page about this hook as been added.
Also update the name of the file to match the hook id.
@codecov
Copy link

codecov bot commented Mar 21, 2023

Codecov Report

Merging #128 (e5a6834) into main (3161d9c) will not change coverage.
The diff coverage is 14.28%.

❗ Current head e5a6834 differs from pull request most recent head f323e21. Consider uploading reports for the commit f323e21 to get more accurate results

@@           Coverage Diff           @@
##             main     #128   +/-   ##
=======================================
  Coverage   65.67%   65.67%           
=======================================
  Files           9        9           
  Lines         472      472           
=======================================
  Hits          310      310           
  Misses        162      162           
Impacted Files Coverage Δ
ci_cd/tasks/update_deps.py 81.09% <14.28%> (ø)

@CasperWA CasperWA added the priority/low Low priority issue/PR label Mar 24, 2023
@Treesarj Treesarj merged commit 9dc7eb7 into main May 15, 2023
@Treesarj Treesarj deleted the cwa/close-24-update-deps-hook branch May 15, 2023 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority/low Low priority issue/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add hook for using the update-deps task
2 participants