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

Update pre-commit config #1482

Merged
merged 1 commit into from
Mar 21, 2023

Conversation

brandonwillard
Copy link
Member

@brandonwillard brandonwillard commented Mar 21, 2023

This PR updates the pre-commit config and addresses a Mypy error involving an import from setuptools._distutils.errors in aesara.link.c.exceptions.

@brandonwillard brandonwillard force-pushed the update-pre-commit branch 2 times, most recently from 18fd090 to 5764b74 Compare March 21, 2023 00:39
@brandonwillard brandonwillard added bug Something isn't working important typing Work related to type checking labels Mar 21, 2023
@codecov
Copy link

codecov bot commented Mar 21, 2023

Codecov Report

Merging #1482 (e137dd8) into main (5268d20) will not change coverage.
The diff coverage is 74.19%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1482   +/-   ##
=======================================
  Coverage   75.02%   75.02%           
=======================================
  Files         194      194           
  Lines       50102    50102           
  Branches    12096    12096           
=======================================
  Hits        37589    37589           
  Misses      10189    10189           
  Partials     2324     2324           
Impacted Files Coverage Δ
aesara/breakpoint.py 55.00% <ø> (ø)
aesara/compile/builders.py 79.16% <ø> (ø)
aesara/compile/function/types.py 79.81% <ø> (ø)
aesara/configdefaults.py 66.20% <ø> (ø)
aesara/gradient.py 77.02% <ø> (ø)
aesara/graph/basic.py 88.24% <ø> (ø)
aesara/graph/rewriting/basic.py 65.03% <0.00%> (ø)
aesara/graph/utils.py 64.84% <ø> (ø)
aesara/ifelse.py 51.14% <ø> (ø)
aesara/link/c/basic.py 87.14% <ø> (ø)
... and 70 more

@brandonwillard brandonwillard merged commit 1da48b5 into aesara-devs:main Mar 21, 2023
@brandonwillard brandonwillard deleted the update-pre-commit branch March 21, 2023 02:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working important typing Work related to type checking
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant