Skip to content

Conversation

@tb65536
Copy link
Collaborator

@tb65536 tb65536 commented Jan 8, 2026

This PR adds a lemma stating that the preimage of a primary ideal under a ring homomorphism remains a primary ideal.


Open in Gitpod

@tb65536 tb65536 added awaiting-author A reviewer has asked the author a question or requested changes. t-algebra Algebra (groups, rings, fields, etc) t-ring-theory Ring theory awaiting-CI This PR doesn't pass CI yet. This label is automatically removed once it does. and removed awaiting-author A reviewer has asked the author a question or requested changes. labels Jan 8, 2026
@github-actions
Copy link

github-actions bot commented Jan 8, 2026

PR summary ee6d0f13b3

Import changes for modified files

Dependency changes

File Base Count Head Count Change
Mathlib.RingTheory.Ideal.IsPrimary 842 845 +3 (+0.36%)
Import changes for all files
Files Import difference
Mathlib.RingTheory.Ideal.IsPrimary 3

Declarations diff

+ IsPrimary.comap

You can run this locally as follows
## summary with just the declaration names:
./scripts/declarations_diff.sh <optional_commit>

## more verbose report:
./scripts/declarations_diff.sh long <optional_commit>

The doc-module for script/declarations_diff.sh contains some details about this script.


No changes to technical debt.

You can run this locally as

./scripts/technical-debt-metrics.sh pr_summary
  • The relative value is the weighted sum of the differences with weight given by the inverse of the current value of the statistic.
  • The absolute value is the relative value divided by the total sum of the inverses of the current values (i.e. the weighted average of the differences).

@github-actions github-actions bot removed the awaiting-CI This PR doesn't pass CI yet. This label is automatically removed once it does. label Jan 9, 2026
Copy link
Member

@erdOne erdOne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!
maintainer merge

@github-actions
Copy link

🚀 Pull request has been placed on the maintainer queue by erdOne.

@leanprover-community-mathlib4-bot leanprover-community-mathlib4-bot added the maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. label Jan 10, 2026
@riccardobrasca
Copy link
Member

Thanks!

bors merge

@leanprover-community-mathlib4-bot leanprover-community-mathlib4-bot added ready-to-merge This PR has been sent to bors. and removed maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. labels Jan 10, 2026
mathlib-bors bot pushed a commit that referenced this pull request Jan 10, 2026
…33781)

This PR adds a lemma stating that the preimage of a primary ideal under a ring homomorphism remains a primary ideal.

Co-authored-by: tb65536 <thomas.l.browning@gmail.com>
@mathlib-bors
Copy link
Contributor

mathlib-bors bot commented Jan 10, 2026

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat(Ideal/IsPrimary): Ideal.isPrimary preserved by Ideal.comap [Merged by Bors] - feat(Ideal/IsPrimary): Ideal.isPrimary preserved by Ideal.comap Jan 10, 2026
@mathlib-bors mathlib-bors bot closed this Jan 10, 2026
@adomani adomani mentioned this pull request Jan 16, 2026
eliasjudin pushed a commit to eliasjudin/mathlib4 that referenced this pull request Jan 18, 2026
…eanprover-community#33781)

This PR adds a lemma stating that the preimage of a primary ideal under a ring homomorphism remains a primary ideal.

Co-authored-by: tb65536 <thomas.l.browning@gmail.com>
goliath-klein pushed a commit to PrParadoxy/mathlib4 that referenced this pull request Jan 24, 2026
…eanprover-community#33781)

This PR adds a lemma stating that the preimage of a primary ideal under a ring homomorphism remains a primary ideal.

Co-authored-by: tb65536 <thomas.l.browning@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge This PR has been sent to bors. t-algebra Algebra (groups, rings, fields, etc) t-ring-theory Ring theory

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants