Skip to content

Eliminate the dependency from orderedzmod to rings_modules_and_algebras#1560

Draft
pi8027 wants to merge 1 commit into
masterfrom
dependency-orderedzmod-algebra
Draft

Eliminate the dependency from orderedzmod to rings_modules_and_algebras#1560
pi8027 wants to merge 1 commit into
masterfrom
dependency-orderedzmod-algebra

Conversation

@pi8027
Copy link
Copy Markdown
Member

@pi8027 pi8027 commented Mar 18, 2026

Motivation for this change

This is one of the things that has to be done before moving orderedzmod to order/.

This change was a part of #1548.

Dependencies
Minimal TODO list
  • added changelog entries with doc/changelog/make-entry.sh
  • added corresponding documentation in the headers
  • tried to abide by the contribution guide
  • this PR contains an optimum number of meaningful commits

See this Checklist for details.

Automatic note to reviewers

Read this Checklist.

@pi8027 pi8027 mentioned this pull request Mar 18, 2026
2 tasks
@pi8027 pi8027 force-pushed the dependency-orderedzmod-algebra branch from 36e9150 to 63f2853 Compare March 19, 2026 11:05
@pi8027 pi8027 force-pushed the dependency-orderedzmod-algebra branch 2 times, most recently from 52815db to 464f090 Compare April 17, 2026 11:49
@pi8027 pi8027 added this to the 2.7.0 milestone Apr 17, 2026
@pi8027
Copy link
Copy Markdown
Member Author

pi8027 commented Apr 17, 2026

I think the main concern here remains to be the fact that the Algebra module does not provide the Theory module, and that we probably don't want to import the entire Algebra module (see #1548). I will add this PR to the topic queue.

@pi8027 pi8027 changed the title Eliminate the dependency from orderedzmod to algebra Eliminate the dependency from orderedzmod to rings_modules_and_algebras Apr 27, 2026
@pi8027 pi8027 mentioned this pull request Apr 28, 2026
14 tasks
@pi8027 pi8027 force-pushed the dependency-orderedzmod-algebra branch from 464f090 to 561d205 Compare May 6, 2026 15:33
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.

1 participant