Pull Request HowTo

[kkremitzki]

c'mon, that's not complicated! That's just two guys writing down the C Major scale
Maybe your mobile browser didn't render the monospaced font nicely?

I understood your example, there was no problem with e.g. the monospaced font rendering. My point was: I understand that there is a problem in the scenario you have constructed. Where is the problem with FreeCAD development that necessitates constructing your example in the first place? What specific problem spurred your investigation? Why not focus on that problem in particular instead of your constructed example? It seems like you're trying to solve for some general case instead of the specific case first, but if you consider all the things people can do to break the directed acyclic graph underlying git, you're going to have an extremely over-engineered fix.

[CADennis]

Thank you for the nice feedback, yorik and kkremitzki. Here are some snippets to hint at the direction how static scans could identify the regressive code jitter. I read your suggestions. And I take them serious. My plan is to avoid déjàvues at manual PR reviews and of course to allow that "master_auto_proposed" automation to no longer break Python3 support.

Code: Select all

git log --graph --all --oneline

* 7d76d70 bc - with love from USA
* 6d6ac96 ah - with love from Germany
* c0c85d1 fga - with love from USA
* 0042ad9 efg - with love from Germany
* 920e32f de - with love from USA
* 0bc90d3 hcd - with love from Germany
* 12b4545 bc - with love from USA
* 7b10573 ga - with love from Germany
* 3b3206d cdef - with love from USA
* 827cff2 let's make music

Code: Select all

user@machine:~/temp/torwaltz/torwaltz_git$ git diff HEAD~3..HEAD~2

diff --git a/song.txt b/song.txt
index 01d7d36..c1475c5 100644
--- a/song.txt
+++ b/song.txt
@@ -1,5 +1,5 @@
cdef
ga
-bc
-de
+hcd
+efg

Code: Select all

user@machine:~/temp/torwaltz/torwaltz_git$ git diff HEAD~1..HEAD

diff --git a/song.txt b/song.txt
index 40e03f8..adffed2 100644
--- a/song.txt
+++ b/song.txt
@@ -1,6 +1,6 @@
cdef
ga
-bc
-de
-fga
+hcd
+efg
+ah

Peace.