slachiewicz commented on PR #3435: URL: https://github.com/apache/maven-surefire/pull/3435#issuecomment-5317267361
Force-pushed one fix: `api`, `developing`, `examples/pojo-test` and `featurematrix` drop the `.vm` suffix, and the first commit was renaming them to `*.md.vm` while the second renamed them again to `*.md` on top of the content rewrite. The combined change fell below rename-detection similarity, so those four showed as add/delete and `git log --follow` stopped at the conversion commit — defeating the reason for splitting the commits at all. `git log --follow` on them reached 1 commit before, and 8, 24, 8 and 14 respectively now. The rename commit now renames each file straight to its final name, and the second commit touches content only — 37 files, all `M`, no add/delete pairs. The resulting tree is byte-identical to what was pushed before, so the site comparison in the description still stands unchanged. Also worth calling out, since it is a deliberate deviation from master rather than part of the 3.5.x delta: `surefire-api/src/site/markdown/index.md` drops the `<th>Term</th><th>Definition</th>` header row that master's conversion introduced. The APT table here has no header row, so master invented one; removing it keeps this page rendering what the 3.5.x site rendered. *This comment was created with AI assistance.* -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
