Skip to content

Minor correct a typo in error msg of Integer.multifactorial#41440

Merged
vbraun merged 1 commit intosagemath:developfrom
MajoranaOedipus:patch-2
Jan 25, 2026
Merged

Minor correct a typo in error msg of Integer.multifactorial#41440
vbraun merged 1 commit intosagemath:developfrom
MajoranaOedipus:patch-2

Conversation

@MajoranaOedipus
Copy link
Copy Markdown
Contributor

if k <= 0:
    raise ValueError("multifactorial only defined for nonpositive k")

nonpositive -> positive

@anneschilling
Copy link
Copy Markdown
Contributor

Looks good to me!

@github-actions
Copy link
Copy Markdown

Documentation preview for this PR (built with commit 0170031; changes) is ready! 🎉
This preview will update shortly after each push to this PR.

vbraun pushed a commit to vbraun/sage that referenced this pull request Jan 17, 2026
sagemathgh-41440: Minor correct a typo in error msg of `Integer.multifactorial`
    
```python
if k <= 0:
    raise ValueError("multifactorial only defined for nonpositive k")
```
`nonpositive` -> `positive`
    
URL: sagemath#41440
Reported by: Majorana Oedipus
Reviewer(s): Travis Scrimshaw
vbraun pushed a commit to vbraun/sage that referenced this pull request Jan 19, 2026
sagemathgh-41440: Minor correct a typo in error msg of `Integer.multifactorial`
    
```python
if k <= 0:
    raise ValueError("multifactorial only defined for nonpositive k")
```
`nonpositive` -> `positive`
    
URL: sagemath#41440
Reported by: Majorana Oedipus
Reviewer(s): Travis Scrimshaw
@vbraun vbraun merged commit 7b12c4d into sagemath:develop Jan 25, 2026
23 of 25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants