Skip to content

[Merged by Bors] - feat(Combinatorics/SimpleGraph/Walk/Operations): clean up append/concat#36705

Closed
SnirBroshi wants to merge 5 commits intoleanprover-community:masterfrom
SnirBroshi:feature/simple-graph/walk/support-perm
Closed

[Merged by Bors] - feat(Combinatorics/SimpleGraph/Walk/Operations): clean up append/concat#36705
SnirBroshi wants to merge 5 commits intoleanprover-community:masterfrom
SnirBroshi:feature/simple-graph/walk/support-perm

Conversation

@SnirBroshi
Copy link
Copy Markdown
Collaborator

@SnirBroshi SnirBroshi commented Mar 16, 2026

_ ++ [_] is the simpNF of List.concat _ _.


This follows the feedback from #35630 (comment) and #35631 (comment)

Open in Gitpod

@SnirBroshi SnirBroshi requested a review from b-mehta March 16, 2026 02:23
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 16, 2026

PR summary 63486b57ca

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference

Declarations diff

+ cons_tail_support
+ dropLast_support_concat

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

## more verbose report:
./scripts/pr_summary/declarations_diff.sh long <optional_commit>

The doc-module for scripts/pr_summary/declarations_diff.sh contains some details about this script.


No changes to technical debt.

You can run this locally as

./scripts/reporting/technical-debt-metrics.sh pr_summary
  • The relative value is the weighted sum of the differences with weight given by the inverse of the current value of the statistic.
  • The absolute value is the relative value divided by the total sum of the inverses of the current values (i.e. the weighted average of the differences).

@github-actions github-actions bot added the t-combinatorics Combinatorics label Mar 16, 2026
@mathlib-merge-conflicts mathlib-merge-conflicts bot added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Mar 19, 2026
@mathlib-merge-conflicts
Copy link
Copy Markdown

This pull request has conflicts, please merge master and resolve them.

@github-actions github-actions bot removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Mar 19, 2026
@ooovi
Copy link
Copy Markdown
Collaborator

ooovi commented Mar 31, 2026

Thank you for the PR! :)

Could you split off the cleanup part into separate chore PRs? It would be very useful for a chore PR to replace all occurrences of List.concat in theorem statements mathlib-wide.

Please also add a descriptive PR description.

@ooovi ooovi added the awaiting-author A reviewer has asked the author a question or requested changes. label Mar 31, 2026
@SnirBroshi SnirBroshi removed the awaiting-author A reviewer has asked the author a question or requested changes. label Mar 31, 2026
@SnirBroshi SnirBroshi changed the title feat(Combinatorics/SimpleGraph/Walks/Operations): permutations of support and cleaning up append/concat feat(Combinatorics/SimpleGraph/Walks/Operations): clean up append/concat Mar 31, 2026
@b-mehta
Copy link
Copy Markdown
Contributor

b-mehta commented Apr 1, 2026

Nice, thank you!

bors merge

@mathlib-triage mathlib-triage bot added the ready-to-merge This PR has been sent to bors. label Apr 1, 2026
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Apr 1, 2026

Merge conflict.

@SnirBroshi SnirBroshi removed the request for review from b-mehta April 1, 2026 13:13
@b-mehta
Copy link
Copy Markdown
Contributor

b-mehta commented Apr 1, 2026

bors d+

@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Apr 1, 2026

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

@mathlib-triage mathlib-triage bot added the delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). label Apr 1, 2026
@SnirBroshi SnirBroshi changed the title feat(Combinatorics/SimpleGraph/Walks/Operations): clean up append/concat feat(Combinatorics/SimpleGraph/Walk/Operations): clean up append/concat Apr 1, 2026
@SnirBroshi
Copy link
Copy Markdown
Collaborator Author

bors merge

mathlib-bors bot pushed a commit that referenced this pull request Apr 1, 2026
…at (#36705)

`_ ++ [_]` is the simpNF of `List.concat _ _`.
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Apr 1, 2026

This PR was included in a batch that was canceled, it will be automatically retried

mathlib-bors bot pushed a commit that referenced this pull request Apr 1, 2026
…at (#36705)

`_ ++ [_]` is the simpNF of `List.concat _ _`.
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Apr 1, 2026

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat(Combinatorics/SimpleGraph/Walk/Operations): clean up append/concat [Merged by Bors] - feat(Combinatorics/SimpleGraph/Walk/Operations): clean up append/concat Apr 1, 2026
@mathlib-bors mathlib-bors bot closed this Apr 1, 2026
aditya-ramabadran pushed a commit to aditya-ramabadran/mathlib4 that referenced this pull request Apr 1, 2026
xroblot pushed a commit to xroblot/mathlib4 that referenced this pull request Apr 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). ready-to-merge This PR has been sent to bors. t-combinatorics Combinatorics

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants