Skip to content

Add support for modern GitFlowNext (Tower) version strings#354

Open
MavapeGZ wants to merge 1 commit into
OpherV:developfrom
MavapeGZ:develop
Open

Add support for modern GitFlowNext (Tower) version strings#354
MavapeGZ wants to merge 1 commit into
OpherV:developfrom
MavapeGZ:develop

Conversation

@MavapeGZ

Copy link
Copy Markdown

The Problem:
Since Git for Windows v2.51.1, the AVH edition of git-flow is no longer included in the installer because the original repository has been unmaintained for years. When users install the modern, actively maintained alternative—GitFlowNext (maintained by Tower)—the plugin fails to initialize and throws a Could not determine git flow version exception. This happens because the plugin strictly required the AVH string in the command output.

The Solution:
This PR updates the version validation logic to accept the modern Tower version strings while continuing to reject the unmaintained Vanilla (0.4.1) version. It also updates the documentation to help users (especially on Windows) avoid installation headaches.

Changes Made:

GitflowVersionTester.java: Modified the isSupportedVersion() method. It now converts the version string to lowercase and checks if it contains avh, tower, or next.

README.md:

Added a specific note for Windows users explaining why they might be seeing "command not found" errors on modern Git installations.

Added GitFlowNext as the recommended installation method, including a Winget command for Windows.

Clarified the version checking instructions to reflect the new supported strings.

Testing:

Verified the plugin successfully initializes without errors in the IntelliJ Sandbox environment while having GitFlowNext installed on the host system.

…nce both of them are working similarlly, no more changes are required on the plugin. Readme was also updated to refer git-flow-next page
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