Skip to content

Comments

remove some deprecated stuff in rings/#41058

Merged
vbraun merged 1 commit intosagemath:developfrom
fchapoton:deprec_rings_2
Oct 27, 2025
Merged

remove some deprecated stuff in rings/#41058
vbraun merged 1 commit intosagemath:developfrom
fchapoton:deprec_rings_2

Conversation

@fchapoton
Copy link
Contributor

after

File: src/sage/rings/polynomial/laurent_polynomial_ring.py ; PR: #35229 closed in 2023
File: src/sage/rings/polynomial/groebner_fan.py ; PR: #33468 closed in 2022
File: src/sage/rings/polynomial/multi_polynomial.pyx ; PR: #32709 closed in 2023
File: src/sage/rings/number_field/number_field.py ; PR: #33386 closed in 2022

📝 Checklist

  • The title is concise and informative.
  • The description explains in detail what this PR is about.

@github-actions
Copy link

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

@cxzhong
Copy link
Contributor

cxzhong commented Oct 17, 2025

LGTM. Thank you.

vbraun pushed a commit to vbraun/sage that referenced this pull request Oct 18, 2025
sagemathgh-41058: remove some deprecated stuff in rings/
    
after

File: src/sage/rings/polynomial/laurent_polynomial_ring.py ; PR: sagemath#35229
closed in 2023
File: src/sage/rings/polynomial/groebner_fan.py ; PR: sagemath#33468 closed in
2022
File: src/sage/rings/polynomial/multi_polynomial.pyx ; PR: sagemath#32709 closed
in 2023
File: src/sage/rings/number_field/number_field.py ; PR: sagemath#33386 closed in
2022

### 📝 Checklist

- [x] The title is concise and informative.
- [x] The description explains in detail what this PR is about.
    
URL: sagemath#41058
Reported by: Frédéric Chapoton
Reviewer(s):
vbraun pushed a commit to vbraun/sage that referenced this pull request Oct 19, 2025
sagemathgh-41058: remove some deprecated stuff in rings/
    
after

File: src/sage/rings/polynomial/laurent_polynomial_ring.py ; PR: sagemath#35229
closed in 2023
File: src/sage/rings/polynomial/groebner_fan.py ; PR: sagemath#33468 closed in
2022
File: src/sage/rings/polynomial/multi_polynomial.pyx ; PR: sagemath#32709 closed
in 2023
File: src/sage/rings/number_field/number_field.py ; PR: sagemath#33386 closed in
2022

### 📝 Checklist

- [x] The title is concise and informative.
- [x] The description explains in detail what this PR is about.
    
URL: sagemath#41058
Reported by: Frédéric Chapoton
Reviewer(s):
vbraun pushed a commit to vbraun/sage that referenced this pull request Oct 20, 2025
sagemathgh-41058: remove some deprecated stuff in rings/
    
after

File: src/sage/rings/polynomial/laurent_polynomial_ring.py ; PR: sagemath#35229
closed in 2023
File: src/sage/rings/polynomial/groebner_fan.py ; PR: sagemath#33468 closed in
2022
File: src/sage/rings/polynomial/multi_polynomial.pyx ; PR: sagemath#32709 closed
in 2023
File: src/sage/rings/number_field/number_field.py ; PR: sagemath#33386 closed in
2022

### 📝 Checklist

- [x] The title is concise and informative.
- [x] The description explains in detail what this PR is about.
    
URL: sagemath#41058
Reported by: Frédéric Chapoton
Reviewer(s):
vbraun pushed a commit to vbraun/sage that referenced this pull request Oct 26, 2025
sagemathgh-41058: remove some deprecated stuff in rings/
    
after

File: src/sage/rings/polynomial/laurent_polynomial_ring.py ; PR: sagemath#35229
closed in 2023
File: src/sage/rings/polynomial/groebner_fan.py ; PR: sagemath#33468 closed in
2022
File: src/sage/rings/polynomial/multi_polynomial.pyx ; PR: sagemath#32709 closed
in 2023
File: src/sage/rings/number_field/number_field.py ; PR: sagemath#33386 closed in
2022

### 📝 Checklist

- [x] The title is concise and informative.
- [x] The description explains in detail what this PR is about.
    
URL: sagemath#41058
Reported by: Frédéric Chapoton
Reviewer(s):
@vbraun vbraun merged commit 26c140b into sagemath:develop Oct 27, 2025
21 of 24 checks passed
@fchapoton fchapoton deleted the deprec_rings_2 branch November 3, 2025 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants