Skip to content

auto-task(leftHandedRep): record TODO to rename Weyl fermion representations - #1311

Merged
jstoobysmith merged 1 commit into
leanprover-community:masterfrom
jstoobysmith:auto-todo-20260629-070120
Jun 29, 2026
Merged

auto-task(leftHandedRep): record TODO to rename Weyl fermion representations#1311
jstoobysmith merged 1 commit into
leanprover-community:masterfrom
jstoobysmith:auto-todo-20260629-070120

Conversation

@jstoobysmith

Copy link
Copy Markdown
Member

Summary

Records a single TODO item in
Physlib/Relativity/Tensors/ComplexTensor/Weyl/Basic.lean, next to the
definition of leftHandedRep, proposing to rename the four Weyl fermion
representations into the namespaces of the modules they act on:

  • leftHandedRepLeftHandedWeyl.rep
  • dualLeftHandedRepDualLeftHandedWeyl.rep
  • rightHandedRepRightHandedWeyl.rep
  • dualRightHandedRepDualRightHandedWeyl.rep

Changes

  • Added public import Physlib.Meta.TODO.Basic.
  • Added one TODO "..." command immediately before leftHandedRep.

No declarations were renamed or implemented; this only records the task. The
project still builds cleanly.


Human review

  • Does the TODO item look correctly placed in the library and an accurate reflection of the input? Yes

…tations

Co-authored-by: Claude <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

Thank you for this PR, which will now be reviewed. If submitting to ./Physlib or ./QuantumInfo, please see our review guidelines if you are not familiar with the process. You should expect a back and forth with a reviewer before your PR is merged. See also that link for how to add appropriate labels to your PR. The PR will also go through a number of automated checks. You can learn more about these here, including how to run them locally.

If you are submitting to ./PhyslibAlpha there will be a lighter review process, though your PR must still pass the automated checks.

If you want to bring attention to this PR, please write a message on this thread of the Lean Zulip.

Important: If a reviewer adds an awaiting-author label to your PR, once you have addressed the review comments, please remove that label by adding a comment with -awaiting-author. This helps us keep track of reviews.

@jstoobysmith
jstoobysmith requested a review from zhikaip June 29, 2026 07:05

@zhikaip zhikaip left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

If this is an auto task I'd rather the PR says more clearly what is being automated, i.e. is it just implementing a user-provided TODO verbatim or did user provide a prompt? (If the latter, maybe the PR summary should include the prompt as well?)

Approved anyway

@jstoobysmith

Copy link
Copy Markdown
Member Author

@zhikaip Yep agreed. I've updated this, and the PR summary now looks like that of #1312, where it includes the original prompt.

The user provides the TODO item, then what is automated is: Fitting the TODO item within the project (i.e. the right place, location, making it fit what is already there etc.)

@jstoobysmith
jstoobysmith merged commit 0c26ee1 into leanprover-community:master Jun 29, 2026
7 checks passed
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