Skip to content

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

Merged
RNKuhns merged 2 commits into
mainfrom
pre-commit-ci-update-config
Jan 26, 2023
Merged

[pre-commit.ci] pre-commit autoupdate#119
RNKuhns merged 2 commits into
mainfrom
pre-commit-ci-update-config

Conversation

@pre-commit-ci
Copy link
Copy Markdown
Contributor

@pre-commit-ci pre-commit-ci Bot commented Jan 17, 2023

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jan 17, 2023

Codecov Report

Merging #119 (a6827e1) into main (250f00f) will not change coverage.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@           Coverage Diff           @@
##             main     #119   +/-   ##
=======================================
  Coverage   78.05%   78.05%           
=======================================
  Files          23       23           
  Lines        1877     1877           
=======================================
  Hits         1465     1465           
  Misses        412      412           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

updates:
- [github.com/asottile/blacken-docs: v1.12.1 → 1.13.0](adamchainz/blacken-docs@v1.12.1...1.13.0)
- [github.com/pycqa/pydocstyle: 6.2.2 → 6.3.0](PyCQA/pydocstyle@6.2.2...6.3.0)
@pre-commit-ci pre-commit-ci Bot force-pushed the pre-commit-ci-update-config branch from b49044d to ff6508e Compare January 24, 2023 02:47
fkiraly added a commit that referenced this pull request Jan 24, 2023
Fixes error in #119.

There were changes in a few of the linters used in `skbase`'s continuous integration process.

- `pydocstyle` moved to the use of `tomli` or `tomllib` for parsing pyproject.toml files (we are going to use `tomli` since `tomllib` is not available in older Python versions)
- `flake8-bugbear` validation B028 was catching cases of manual quotes in string formatting. It caused #119 to fail. `flake8-bugbear` has since moved this to an optional validation. This enables that validation, along with two others. And fixes the resulting items flagged by the CI.

  - Note that B028 was changed to B907. And optional validations B03 and B904 were also turned on.  `flake8-bugbear` [readme](https://github.com/PyCQA/flake8-bugbear) has a description. This PR includes fixes issues identified by B904.
@RNKuhns RNKuhns merged commit 36167b7 into main Jan 26, 2023
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.

2 participants