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

[MNT] convert black extend-exclude parameter to single string #207

Merged
merged 2 commits into from
Jul 13, 2023

Conversation

fkiraly
Copy link
Contributor

@fkiraly fkiraly commented Jul 12, 2023

In #205, pre-commit CI seems to complain that the extend-exclude parameter is not followed by a string (but a list of strings).

Attempting to fix this by changing the list of strings into a single string tha should hopefully comply with the expected syntax.

@fkiraly fkiraly added the maintenance Continuous integration, unit testing & package distribution label Jul 12, 2023
@codecov-commenter
Copy link

codecov-commenter commented Jul 12, 2023

Codecov Report

Merging #207 (628d8bd) into main (34e8679) will not change coverage.
The diff coverage is n/a.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@           Coverage Diff           @@
##             main     #207   +/-   ##
=======================================
  Coverage   82.41%   82.41%           
=======================================
  Files          43       43           
  Lines        2832     2832           
=======================================
  Hits         2334     2334           
  Misses        498      498           

@fkiraly fkiraly merged commit 1722ec5 into main Jul 13, 2023
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Continuous integration, unit testing & package distribution
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants