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

Feature request: git-together pair ab cd #19

Closed
professor opened this issue Jun 1, 2017 · 4 comments
Closed

Feature request: git-together pair ab cd #19

professor opened this issue Jun 1, 2017 · 4 comments

Comments

@professor
Copy link
Contributor

As a developer, I expect to easily set who is currently paired on my system.

When I type 'git-together pair ab cd'
Then I expect them to be paired

Sidenote: I'm worried about this confusing people who have been using git pair and them not realizing that something different is going on. I'm not sure how to handle this.

@kejadlen
Copy link
Owner

kejadlen commented Jun 5, 2017

That would be my worry as well, since the two tools have quite different behavior. Maybe using a git alias for people that want to keep pair as the subcommand would be an acceptable solution?

@professor
Copy link
Contributor Author

What about 'git together ab cd' ?

@professor
Copy link
Contributor Author

I'm still thinking about this topic. I'd like to +1 the git together ab cd suggestion.

Here is one more reason. At the end of the workstation-setup script, we print out...

You can use 'git pair' (enabled) 'git duet' (enabled) or 'git with' aka git-together (installed but not enabled)
To enable git together
git=git-together' >> ~/.bash_profile

I'd love for that to say

You can use 'git pair' (enabled) 'git duet' (enabled) or 'git together' (installed but not enabled)

In other words, the name of the tool should also be a command. If we want only git with to work, then the name of the tool and repo should be git with instead of git together

@kejadlen
Copy link
Owner

Makes sense to me!

kejadlen pushed a commit that referenced this issue Oct 20, 2018
* Rustfmt

* Rustfmt

* Implementing --global flag

Signed-off-by: Andres Medina <[email protected]>

* Fix --global option implementation

Note: relies on global .gitconfig being copied vs. symlinked due to bug
in libgit2.

Signed-off-by: Andres Medina <[email protected]>

* Rustfmt all the things

via ```cargo fmt```

* Add rustfmt to the travis build step

Using http://johannh.me/blog/rustfmt-ci.html as a guide.

* Adding a license per the open issue

MIT because ¯\_(ツ)_/¯

* Add more detailed build instructions

* small clippy refactoring

* oops, I broke it

* Tweak `with` options

- Default to showing current pair
- `--list` to show all authors
- `--version` to show the version

Fixes #17

* Remove rustfmt from CI

* Remove misleading TODOs from README

* Add `together` as a trigger

Fixes #19

* Add directions on how to install git-together

* Shows how to use a global .git-together or one per project

* Moved list of signoff commands into GitTogether

* Added support for aliases. Fixes #8.

Supported aliases are pulled from the git-together.aliases config
variable, which (if set) should be a comma-separated list of aliases
e.g. "ci,rv,m".

Signed-off-by: Steve Gravrock <[email protected]>

* fixes from clippy

* rustfmt

* start keeping a changelog

* Add linux target to environment list.

* v0.1.0-alpha.15

* remove `advanced_slice_patterns`

* v0.1.0-alpha.16

* Updates version of git2 to 0.7

Authored-by: Todd Sedano <[email protected]>

* Implment --clear and linter fixes

Signed-off-by: Andres Medina <[email protected]>

* cargo fmt

Signed-off-by: Scott Windsor <[email protected]>

* Use --global for all commands

Signed-off-by: Andres Medina <[email protected]>

* Also clear users when clearing active user

Signed-off-by: Andres Medina <[email protected]>

* Update README for --global

Signed-off-by: Scott Windsor <[email protected]>

* Remove commented-out `rustfmt` installation

* Remove commented out code
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

No branches or pull requests

2 participants