Skip to content

installer: refine git pull behavior text to prevent misconfiguration#682

Open
JZBAKH wants to merge 1 commit intogit-for-windows:mainfrom
JZBAKH:text-refactor-git-pull
Open

installer: refine git pull behavior text to prevent misconfiguration#682
JZBAKH wants to merge 1 commit intogit-for-windows:mainfrom
JZBAKH:text-refactor-git-pull

Conversation

@JZBAKH
Copy link

@JZBAKH JZBAKH commented Mar 13, 2026

Description

The previous phrasing of the git pull behavior defaults could lead to unintended configurations of the pull.rebase vs pull.ff parameters in the global .gitconfig. This PR refactors the text to introduce strict mechanical representations of the git state machine. The goal is to reduce ambiguity concerning commit graph resolution and align the UI text with current upstream defaults.

Technical Scope

  • Textual and UI Layout Update: This modification strictly targets the strings passed to the CreatePage and CreateRadioButton functions for the GP_GitPullMerge, GP_GitPullRebase, and GP_GitPullFFOnly options.
  • Added Label: A standard TLabel (LblInfo) was added at the bottom of the page to clarify how Git handles file conflicts.
  • Logic Intact: No underlying Pascal Script conditional logic, registry parsing (ReplayChoice), or silent install vectors were altered. The internal component IDs remain untouched.

Visual Regression Testing

Before After
git_pull_behavior git_pull_behavior_2

Signed-off-by: Jason Zbakh jzbakh@gmail.com

The previous phrasing of the git pull behavior options lacked explicit
mechanical descriptions, potentially leading to unintended configurations
of the pull.rebase and pull.ff parameters.

This commit refactors the descriptive strings in install.iss to use
explicit commit graph representations (e.g., [Local] + [Remote]). It
also updates the page subtitle and appends a clarification note about
line conflicts to prevent working tree corruption and align with
upstream defaults.

Signed-off-by: Jason Zbakh <jzbakh@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant