Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Merged by Bors] - doc: Mark named theorems including Compactness Theorem, Halting Problem, Tarski Vaught, Łoś #8743

Closed
wants to merge 8 commits into from

Conversation

hmonroe
Copy link
Collaborator

@hmonroe hmonroe commented Nov 30, 2023


Open in Gitpod

@hmonroe hmonroe added awaiting-review The author would like community review of the PR easy < 20s of review time. See the lifecycle page for guidelines. documentation Improvements or additions to documentation labels Nov 30, 2023
Copy link
Member

@eric-wieser eric-wieser left a comment

Choose a reason for hiding this comment

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

bors d+

Don't forget to fix the long line.

@mathlib-bors
Copy link

mathlib-bors bot commented Nov 30, 2023

✌️ hmonroe can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

@leanprover-community-mathlib4-bot leanprover-community-mathlib4-bot added delegated and removed awaiting-review The author would like community review of the PR labels Nov 30, 2023
@@ -154,8 +154,8 @@ theorem realize_formula_cast {β : Type*} (φ : L.Formula β) (x : β → ∀ a,
exact congr rfl (Subsingleton.elim _ _)
#align first_order.language.ultraproduct.realize_formula_cast FirstOrder.Language.Ultraproduct.realize_formula_cast

/-- Łoś's Theorem : A sentence is true in an ultraproduct if and only if the set of structures it is
true in is in the ultrafilter. -/
/-- **Łoś's Theorem**: A sentence is true in an ultraproduct if and only if the set of structures

Choose a reason for hiding this comment

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

[lint-style] reported by reviewdog 🐶

Suggested change
/-- **Łoś's Theorem**: A sentence is true in an ultraproduct if and only if the set of structures
/-- **Łoś's Theorem**: A sentence is true in an ultraproduct if and only if the set of structures

Comment on lines 157 to 158
/-- **Łoś's Theorem**: A sentence is true in an ultraproduct if and only if the set of structures
it is true in is in the ultrafilter. -/

Choose a reason for hiding this comment

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

[lint-style] reported by reviewdog 🐶

Suggested change
/-- **Łoś's Theorem**: A sentence is true in an ultraproduct if and only if the set of structures
it is true in is in the ultrafilter. -/
/-- **Łoś's Theorem**: A sentence is true in an ultraproduct if and only if the set of structures
it is true in is in the ultrafilter. -/

@hmonroe
Copy link
Collaborator Author

hmonroe commented Nov 30, 2023

Long line and trailing spaces are fixed. bors is asking me to approve which may not be correct

bors d+

Don't forget to fix the long line.

@@ -230,10 +230,12 @@ theorem rice₂ (C : Set Code) (H : ∀ cf cg, eval cf = eval cg → (cf ∈ C
exact ⟨by infer_instance, Computable.const _⟩ }⟩
#align computable_pred.rice₂ ComputablePred.rice₂

/-- The Halting problem is recursively enumerable */
Copy link
Member

Choose a reason for hiding this comment

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

This is a syntax error

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Apologies I should not have applied the Awaiting Review label until checks were complete—I am looking at the checks. I will be making commits from VS Code going forward

@hmonroe
Copy link
Collaborator Author

hmonroe commented Dec 4, 2023

bors r+

mathlib-bors bot pushed a commit that referenced this pull request Dec 4, 2023
@mathlib-bors
Copy link

mathlib-bors bot commented Dec 5, 2023

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title doc: Mark named theorems including Compactness Theorem, Halting Problem, Tarski Vaught, Łoś [Merged by Bors] - doc: Mark named theorems including Compactness Theorem, Halting Problem, Tarski Vaught, Łoś Dec 5, 2023
@mathlib-bors mathlib-bors bot closed this Dec 5, 2023
@mathlib-bors mathlib-bors bot deleted the hmonroe-patch-1 branch December 5, 2023 00:40
awueth pushed a commit that referenced this pull request Dec 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
delegated documentation Improvements or additions to documentation easy < 20s of review time. See the lifecycle page for guidelines.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants