Github conflicts
Posted: Thu Oct 29, 2020 1:36 pm
A few weeks ago, I developed some enhancements to MonthCalendar, Grid, and IsVista(), as described here: http://hmgforum.com/viewtopic.php?f=5&t=6607
I put these into an fork https://github.com/kcarmody/HMG of the main HMG Github source https://github.com/HMG-Official/, and submitted a pull request https://github.com/HMG-Official/HMG/pull/2 .
When I submitted this pull request, Github said there were no conflicts with the master, but now it is saying that there are conflicts in 4 files. It appears that these conflicts are a result of merging some changes from another pull request that preceded mine, from a user named asistex.
Most of the changes from that pull request were labeled "removed ending spces" or "Update <filename>". I checked a few of these changes and found that this project had not only removed spaces from ends of lines, but also removed variables that were declared but never used, and removed variable initializations that were not needed. Every file in SOURCE was changed.
I'm not sure why these changes were all made, as they do not affect any features and have almost no effect on performance. But in any event, they have made it harder to merge my code into the main HMG code. My code adds several new features.
I notice that the merge from asistex's code was done on 23 October, and no action has yet been taken on my code, which is next in line. I have a question for whoever does the merging. Will you go ahead and merge my code in the HMG master, or are you waiting for me to resolve the conflicts?
I put these into an fork https://github.com/kcarmody/HMG of the main HMG Github source https://github.com/HMG-Official/, and submitted a pull request https://github.com/HMG-Official/HMG/pull/2 .
When I submitted this pull request, Github said there were no conflicts with the master, but now it is saying that there are conflicts in 4 files. It appears that these conflicts are a result of merging some changes from another pull request that preceded mine, from a user named asistex.
Most of the changes from that pull request were labeled "removed ending spces" or "Update <filename>". I checked a few of these changes and found that this project had not only removed spaces from ends of lines, but also removed variables that were declared but never used, and removed variable initializations that were not needed. Every file in SOURCE was changed.
I'm not sure why these changes were all made, as they do not affect any features and have almost no effect on performance. But in any event, they have made it harder to merge my code into the main HMG code. My code adds several new features.
I notice that the merge from asistex's code was done on 23 October, and no action has yet been taken on my code, which is next in line. I have a question for whoever does the merging. Will you go ahead and merge my code in the HMG master, or are you waiting for me to resolve the conflicts?