Skip to content

Commit 0ecad0f

Browse files
author
Release Manager
committed
gh-40348: Fix a small typo change `bool` to `positive`. And reformat some `%` to f-string. I don't have gap3 to test, but this looks simple enough that surely it can't go wrong (?) ### 📝 Checklist <!-- Put an `x` in all the boxes that apply. --> - [ ] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [ ] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies <!-- List all open PRs that this PR logically depends on. For example, --> <!-- - #12345: short description why this is a dependency --> <!-- - #34567: ... --> URL: #40348 Reported by: user202729 Reviewer(s): Frédéric Chapoton
2 parents 02fe335 + 91737dd commit 0ecad0f

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/sage/combinat/root_system/reflection_group_element.pyx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -812,10 +812,10 @@ cdef class RealReflectionGroupElement(ComplexReflectionGroupElement):
812812
False
813813
"""
814814
if not isinstance(positive, bool):
815-
raise TypeError("%s is not a boolean" % (bool))
815+
raise TypeError(f"{positive} is not a boolean")
816816

817817
if i not in self._parent.index_set():
818-
raise ValueError("the given index %s is not in the index set" % i)
818+
raise ValueError(f"the given index {i} is not in the index set")
819819

820820
negative = not positive
821821

0 commit comments

Comments
 (0)