Skip to content

[Merged by Bors] - chore(PULL_REQUEST_TEMPLATE.md): add template for moves and deletions#14559

Closed
jcommelin wants to merge 4 commits intomasterfrom
jcommelin-patch-1
Closed

[Merged by Bors] - chore(PULL_REQUEST_TEMPLATE.md): add template for moves and deletions#14559
jcommelin wants to merge 4 commits intomasterfrom
jcommelin-patch-1

Conversation

@jcommelin
Copy link
Copy Markdown
Member


Open in Gitpod

@github-actions
Copy link
Copy Markdown

github-actions bot commented Jul 9, 2024

PR summary 3d0ea229c3

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference

Declarations diff

No declarations were harmed in the making of this PR! 🐙

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

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

@kim-em
Copy link
Copy Markdown
Contributor

kim-em commented Jul 9, 2024

Is the intention that a bot will fill this in?

@jcommelin
Copy link
Copy Markdown
Member Author

Maybe not directly filling in, but certainly the intention is that a bot will generate a first draft that can be copy-pasta-edited.

@adomani
Copy link
Copy Markdown
Contributor

adomani commented Jul 9, 2024

In particular, the pairs of declarations that decl_diff matches, are good candidates for Moves and the ones that have an unmatched - are good candidates for Deletions.

@kim-em kim-em added the awaiting-author A reviewer has asked the author a question or requested changes. label Jul 9, 2024
Co-authored-by: damiano <adomani@gmail.com>
@jcommelin jcommelin removed the awaiting-author A reviewer has asked the author a question or requested changes. label Jul 10, 2024
@adomani
Copy link
Copy Markdown
Contributor

adomani commented Jul 10, 2024

I looked briefly into this and I am only seeing hacky ways of getting a PR template that depends on the PR itself.

As far as I understand, GitHub simply reads the file that this PR is editing and uses that as a template. So, this file would have had to be edited between pushing and creating a PR. For instance, the very first action upon pushing could be to compute the decl_diff, modify the PR_template file and then have it ready by the time you actually open the PR!

I wonder if not git-indexing this file and simply regenerating it on every push would give the "personalised touch" that we want.

@kim-em
Copy link
Copy Markdown
Contributor

kim-em commented Jul 10, 2024

I think if we develop a bot that can contribute answers here, it should just run after the PR is open. We can add a note to the template saying "You don't need to fill this in immediately: a bot will edit the PR top comment shortly with a first approximation, and you should check that it looks right."

@kim-em
Copy link
Copy Markdown
Contributor

kim-em commented Jul 10, 2024

bors merge

@github-actions github-actions bot added the ready-to-merge This PR has been sent to bors. label Jul 10, 2024
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Jul 10, 2024

Build failed (retrying...):

@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Jul 10, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title chore(PULL_REQUEST_TEMPLATE.md): add template for moves and deletions [Merged by Bors] - chore(PULL_REQUEST_TEMPLATE.md): add template for moves and deletions Jul 10, 2024
@mathlib-bors mathlib-bors bot closed this Jul 10, 2024
@mathlib-bors mathlib-bors bot deleted the jcommelin-patch-1 branch July 10, 2024 18:31
@adomani adomani mentioned this pull request Aug 1, 2024
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants