Skip to content

Add GitRemotes() and GitRemote() aliases#192

Merged
pascalberger merged 2 commits into
cake-contrib:developfrom
ap0llo:add-git-remote-aliases
Dec 14, 2024
Merged

Add GitRemotes() and GitRemote() aliases#192
pascalberger merged 2 commits into
cake-contrib:developfrom
ap0llo:add-git-remote-aliases

Conversation

@ap0llo

@ap0llo ap0llo commented Jul 24, 2024

Copy link
Copy Markdown
Contributor

Add aliases to retrieve information about a git repository's remotes.

  • GitRemotes(): List all of a repository's remotes
  • GitRemote(): Get a specific remote by name (or null if no such remote exists)

It was already possible to get the remotes indirectly via the GitBranchCurrentAlias() but I found this counterintuitive and hard to discover.

I think it makes sense to have dedicated aliases for working with remotes as this git functionality is not directly tied to any branch.

Fixes partially #147

@ap0llo ap0llo changed the title Add GitRemotes() and GitRemote() aliases Add GitRemotes() and GitRemote() aliases (#147) Aug 9, 2024
Add aliases to retrieve information about a git repository's remotes.

GitRemotes(): List all of a repository's remotes
GitRemote(): Get a specific remote by name
@pascalberger pascalberger force-pushed the add-git-remote-aliases branch from 13ae461 to 8b43ab0 Compare December 14, 2024 07:44
@pascalberger pascalberger force-pushed the add-git-remote-aliases branch from 31a0c56 to 302c594 Compare December 14, 2024 08:11

@pascalberger pascalberger left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

I updated code to newer .NET versions which is now possible, made smaller improvements to docs and added test cases.

@pascalberger pascalberger changed the title Add GitRemotes() and GitRemote() aliases (#147) Add GitRemotes() and GitRemote() aliases Dec 14, 2024
@pascalberger pascalberger merged commit 746f1a6 into cake-contrib:develop Dec 14, 2024
@pascalberger

Copy link
Copy Markdown
Member

@ap0llo your changes have been merged, thanks for your contribution 👍

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.

2 participants