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

make format.python is broken #1142

Closed
dalf opened this issue Apr 22, 2022 · 2 comments · Fixed by #1185
Closed

make format.python is broken #1142

dalf opened this issue Apr 22, 2022 · 2 comments · Fixed by #1185
Labels
bug Something isn't working

Comments

@dalf
Copy link
Member

dalf commented Apr 22, 2022

make format.python runs

black --target-version py37 --line-length 120 --skip-string-normalization --exclude searx/static,searx/languages.py --include searxng.msg searx searxng_extra tests

doesn't fix the code

it works without --include searxng.msg:
black --target-version py37 --line-length 120 --skip-string-normalization --exclude searx/static,searx/languages.py searx searxng_extra tests

@dalf dalf added the bug Something isn't working label Apr 22, 2022
return42 added a commit to return42/searxng that referenced this issue Apr 23, 2022
--include TEXT

  A regular expression that matches files and directories that should be
  included on recursive searches. An empty value means all files are included
  regardless of the name.  ...  Exclusions are calculated first, inclusions
  later.  [default: \.pyi?$]

Closes: searxng#1142
Signed-off-by: Markus Heiser <markus.heiser@darmarit.de>
@return42
Copy link
Member

Sorry, my fail .. fixed in #1143 / never seen that there is only one file tested:

grafik

such blinky messages are nice on colorized utf-8 terminals but hard to read on dump terminals.

grafik

@dalf
Copy link
Member Author

dalf commented Apr 23, 2022

such blinky messages are nice on colorized utf-8 terminals but hard to read on dump terminals.

See:

return42 added a commit to return42/searxng that referenced this issue Apr 24, 2022
--include TEXT

  A regular expression that matches files and directories that should be
  included on recursive searches. An empty value means all files are included
  regardless of the name.  ...  Exclusions are calculated first, inclusions
  later.  [default: \.pyi?$]

Closes: searxng#1142
Signed-off-by: Markus Heiser <markus.heiser@darmarit.de>
return42 added a commit to return42/searxng that referenced this issue Apr 30, 2022
--include TEXT

  A regular expression that matches files and directories that should be
  included on recursive searches. An empty value means all files are included
  regardless of the name.  ...  Exclusions are calculated first, inclusions
  later.  [default: \.pyi?$]

Closes: searxng#1142
Signed-off-by: Markus Heiser <markus.heiser@darmarit.de>
return42 added a commit to return42/searxng that referenced this issue May 3, 2022
--include TEXT

  A regular expression that matches files and directories that should be
  included on recursive searches. An empty value means all files are included
  regardless of the name.  ...  Exclusions are calculated first, inclusions
  later.  [default: \.pyi?$]

Closes: searxng#1142
Signed-off-by: Markus Heiser <markus.heiser@darmarit.de>
return42 added a commit to return42/searxng that referenced this issue May 7, 2022
--include TEXT

  A regular expression that matches files and directories that should be
  included on recursive searches. An empty value means all files are included
  regardless of the name.  ...  Exclusions are calculated first, inclusions
  later.  [default: \.pyi?$]

Closes: searxng#1142
Signed-off-by: Markus Heiser <markus.heiser@darmarit.de>
return42 added a commit to return42/searxng that referenced this issue May 7, 2022
--include TEXT

  A regular expression that matches files and directories that should be
  included on recursive searches. An empty value means all files are included
  regardless of the name.  ...  Exclusions are calculated first, inclusions
  later.  [default: \.pyi?$]

Closes: searxng#1142
Signed-off-by: Markus Heiser <markus.heiser@darmarit.de>
return42 added a commit to return42/searxng that referenced this issue May 7, 2022
--include TEXT

  A regular expression that matches files and directories that should be
  included on recursive searches. An empty value means all files are included
  regardless of the name.  ...  Exclusions are calculated first, inclusions
  later.  [default: \.pyi?$]

Closes: searxng#1142
Signed-off-by: Markus Heiser <markus.heiser@darmarit.de>
return42 added a commit to return42/searxng that referenced this issue May 7, 2022
--include TEXT

  A regular expression that matches files and directories that should be
  included on recursive searches. An empty value means all files are included
  regardless of the name.  ...  Exclusions are calculated first, inclusions
  later.  [default: \.pyi?$]

Closes: searxng#1142
Signed-off-by: Markus Heiser <markus.heiser@darmarit.de>
@dalf dalf closed this as completed in #1185 May 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
2 participants