Skip to content

fix: move set_option inside @[expose] public section in Lemmas#1747

Closed
kim-em wants to merge 1 commit intomainfrom
fix-set-option-expose-v2
Closed

fix: move set_option inside @[expose] public section in Lemmas#1747
kim-em wants to merge 1 commit intomainfrom
fix-set-option-expose-v2

Conversation

@kim-em
Copy link
Copy Markdown
Collaborator

@kim-em kim-em commented Apr 1, 2026

This PR moves set_option linter.deprecated false from before @[expose] public section to inside it in Batteries/Data/String/Lemmas.lean.

The set_option ... in @[expose] public section pattern (introduced in #1736) causes the section's declarations to not be exported via normal import, making constants like utf8Len_cons, rawEndPos_ofList, and hasNext_cons_addChar invisible to downstream consumers (including Mathlib).

🤖 Prepared with Claude Code

…ic section`

`set_option ... in @[expose] public section` causes the section's
declarations to not be exported via normal `import`. Moving the
`set_option` inside the section fixes this.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions github-actions bot added the awaiting-review This PR is ready for review; the author thinks it is ready to be merged. label Apr 1, 2026
mathlib-nightly-testing bot added a commit to leanprover-community/mathlib4-nightly-testing that referenced this pull request Apr 1, 2026
@fgdorais
Copy link
Copy Markdown
Collaborator

fgdorais commented Apr 1, 2026

I reverted #1736 and added this change in the new version at #1750

@fgdorais fgdorais closed this Apr 1, 2026
@github-actions github-actions bot removed the awaiting-review This PR is ready for review; the author thinks it is ready to be merged. label Apr 1, 2026
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.

2 participants