Skip to content

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

Merged
fkiraly merged 2 commits into
mainfrom
extend-exclude-str
Jul 13, 2023
Merged

[MNT] convert black extend-exclude parameter to single string#207
fkiraly merged 2 commits into
mainfrom
extend-exclude-str

Conversation

@fkiraly
Copy link
Copy Markdown
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
Copy Markdown

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
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