Skip to content

Conversation

mstv
Copy link
Member

@mstv mstv commented Dec 4, 2023

Fixes #11424

Proposed changes

  • Ignore fetch/push options in GetRemotesAsync

Screenshots

N/A

Test methodology

  • extended testcase

Merge strategy

I agree that the maintainer squash merge this PR (if the commit message is clear).


✒️ I contribute this code under The Developer Certificate of Origin.

@mstv mstv self-assigned this Dec 4, 2023
@mstv mstv changed the title Ignore fetch/push options in GetRemotesAsync Ignore fetch/push options in configuration of remote repos Dec 4, 2023
Copy link
Member

@gerhardol gerhardol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 due to comments, not opposing merging

@mstv mstv merged commit 0ca94a9 into gitextensions:release/4.0 Dec 5, 2023
@mstv mstv deleted the feature/i11424_remotes_with_options branch December 5, 2023 21:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants