[Merged by Bors] - refactor(Combinatorics/SimpleGraph): make the vertex argument explicit in Walk.rotate

[YaelDillies]

It is annoying for it not to appear in the infoview. In general, no implicit data argument to a definition should be inferrable only through proof arguments.

---

I could further remove the proof obligation by making Walk.rotate return the original walk if the vertex is not in the support. What do you think?

[github-actions]

PR summary 2352820370

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/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).

[b-mehta]

I could further remove the proof obligation by making Walk.rotate return the original walk if the vertex is not in the support. What do you think?

Could do, but I think the current PR is an improvement regardless, so let's investigate this in follow-up PRs. Thanks!

bors merge

[mathlib-bors]

Pull request successfully merged into master.

Build succeeded:

• ci (staging) / Build
• ci (staging) / Lint style
• ci (staging) / Post-CI job
• ci (staging) / Post-Build Step