Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add GitRemotes() and GitRemote() aliases #192

Merged
merged 2 commits into from
Dec 14, 2024

Conversation

ap0llo
Copy link
Contributor

@ap0llo ap0llo commented Jul 24, 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 (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
Copy link
Member

@pascalberger pascalberger left a comment

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
6 checks passed
@pascalberger
Copy link
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