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

[pre-commit.ci] pre-commit autoupdate #203

Merged
merged 4 commits into from
Jun 3, 2024
Merged

Conversation

Copy link

codecov bot commented Mar 25, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.63%. Comparing base (2d09424) to head (b8f92da).
Report is 1 commits behind head on main.

Current head b8f92da differs from pull request most recent head 24bdab8

Please upload reports for the commit 24bdab8 to get more accurate results.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #203      +/-   ##
==========================================
- Coverage   99.63%   99.63%   -0.01%     
==========================================
  Files           9        9              
  Lines        1093     1092       -1     
==========================================
- Hits         1089     1088       -1     
  Misses          4        4              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch 2 times, most recently from f9148da to 7866f34 Compare April 15, 2024 17:24
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 7866f34 to d7a8db3 Compare April 29, 2024 17:26
updates:
- [github.com/pre-commit/pre-commit-hooks: v4.5.0 → v4.6.0](pre-commit/pre-commit-hooks@v4.5.0...v4.6.0)
- [github.com/codespell-project/codespell: v2.2.6 → v2.3.0](codespell-project/codespell@v2.2.6...v2.3.0)
- [github.com/asottile/pyupgrade: v3.15.1 → v3.15.2](asottile/pyupgrade@v3.15.1...v3.15.2)
- [github.com/psf/black: 24.3.0 → 24.4.2](psf/black@24.3.0...24.4.2)
- [github.com/pre-commit/mirrors-mypy: v1.9.0 → v1.10.0](pre-commit/mirrors-mypy@v1.9.0...v1.10.0)
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from d7a8db3 to 5d66d6c Compare May 27, 2024 17:18
jim-smith and others added 3 commits May 30, 2024 23:59
Signed-off-by: Jim-smith <jim-smith@users.noreply.github.com>
Signed-off-by: Jim-smith <jim-smith@users.noreply.github.com>
@jim-smith jim-smith requested a review from rpreen May 30, 2024 23:12
@jim-smith
Copy link
Contributor

linting is complaining that the method create_dataframe (starts line 1360ish)
in
acro_tables
line 1375 assume that th tow sub-data frames (index and columns) have been successfully created.
But they rely on both the methods params being either a list or a series.
Could with some error handling

@jim-smith jim-smith merged commit d8e9b2e into main Jun 3, 2024
3 of 4 checks passed
@jim-smith jim-smith deleted the pre-commit-ci-update-config branch June 3, 2024 11:03
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.

1 participant