Skip to content

wrap long lines in Main declarations: in project card - #711

Open
loefflerd wants to merge 1 commit into
LeanPool:mainfrom
loefflerd:CI-wrap-main-decls
Open

loefflerd wants to merge 1 commit into
LeanPool:mainfrom
loefflerd:CI-wrap-main-decls

Conversation

@loefflerd

Copy link
Copy Markdown

At present, CI enforces that the Main declarations: line in the project card matches the autogenerated one from the YAML file, but it also enforces that the line can't be over 100 chars long. This change adjusts the CI scripts to wrap the line.

(Implemented by ChatGPT 6.1 Sol Medium using Codex)

@github-actions

Copy link
Copy Markdown
Contributor

Proof profile (new / modified Lean files)

No added or modified Lean files under LeanPool/ in this PR. No compile-cost measurements are needed.

Advisory only — never blocks merge.

@greptile-apps

greptile-apps Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

Advisory cross-file review against Lean Pool's supervisory rubric. This is not by itself a significance, source-verification, or merge verdict.

RetriggerView in Greptile

[Low impact] The PR appears safe to merge; no actionable issues were found.

Summary

Wraps generated Main declarations: text at 100 characters, with indented continuation lines and whole identifiers.

  • Long project declaration lists now wrap in generated cards.

Reviews (1) · Last reviewed commit: "wrap long lines in `Main declarations: `..." · Reviewed by Greptile

This branch has not been deployed

No deployments
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