[Bug 2351032] Review Request: python-tatsu - Python parser generator from grammars in a variation of EBNF

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=2351032

Ben Beasley <code@xxxxxxxxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |code@xxxxxxxxxxxxxxxxxx



--- Comment #3 from Ben Beasley <code@xxxxxxxxxxxxxxxxxx> ---
This fails to build with:

+ /usr/bin/python3 -m tox --current-env --assert-config -q --recreate -e py313
tatsu/bootstrap.py:12:1: RUF100 [*] Unused `noqa` directive (unused: `C405`)
   |
10 | #  the file is generated.
11 |
12 | # ruff: noqa: C405, COM812, I001, F401, PLR1702, PLC2801, SIM117
   | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ RUF100
13 |
14 | import sys
   |
   = help: Remove unused `noqa` directive
tatsu/tool.py:5:1: RUF100 [*] Unused `noqa` directive (unused: `PLR0912`)
  |
3 | the described language.
4 | """
5 | # ruff: noqa: PLR0912
  | ^^^^^^^^^^^^^^^^^^^^^ RUF100
6 | from __future__ import annotations
  |
  = help: Remove unused `noqa` directive
test/grammar/semantics_test.py:1:1: RUF100 [*] Unused `noqa` directive
(non-enabled: `S102`)
  |
1 | # ruff: noqa: S102
  | ^^^^^^^^^^^^^^^^^^ RUF100
2 | import unittest
  |
  = help: Remove unused `noqa` directive
test/parser_equivalence_test.py:1:1: RUF100 [*] Unused `noqa` directive
(non-enabled: `S102`)
  |
1 | # ruff: noqa: S102
  | ^^^^^^^^^^^^^^^^^^ RUF100
2 | import types
  |
  = help: Remove unused `noqa` directive
Found 4 errors.
[*] 4 fixable with the `--fix` option.
py313: exit 1 (0.04 seconds)
/builddir/build/BUILD/python-tatsu-5.13.1-build/TatSu-5.13.1> ruff check
--preview tatsu test examples pid=817

You should patch out the ruff and mypy invocations from tox.ini as prescribed
by https://docs.fedoraproject.org/en-US/packaging-guidelines/Python/#_linters,
or (easier) bypass tox, drop -t from %pyproject_requires, add a manual
BuildRequires on pytest and just run the test suite with %pytest.


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
https://bugzilla.redhat.com/show_bug.cgi?id=2351032

Report this comment as SPAM: https://bugzilla.redhat.com/enter_bug.cgi?product=Bugzilla&format=report-spam&short_desc=Report%20of%20Bug%202351032%23c3

-- 
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux