Skip to content

Make switch command a bit more helpful #133

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Feb 27, 2017
Merged

Make switch command a bit more helpful #133

merged 2 commits into from
Feb 27, 2017

Conversation

@katrinleinweber
Copy link
Contributor

@katrinleinweber katrinleinweber commented Feb 23, 2017 •
edited
Loading

fixes #128

./setup.py test returns Ran 111 tests in 351.188s, then OK and then many of these:

Exception AttributeError: "'NoneType' object has no attribute 'git_remote_free'" in 
<bound method Remote.__del__ of <pygit2.remote.Remote object at 0x10b6a5850
>> ignored

I guess Travis will tell...

@katrinleinweber katrinleinweber changed the title Make switch command a bit more helpful (fixes #128) Make switch command a bit more helpful Feb 23, 2017
spderosso
spderosso reviewed Feb 27, 2017
View reviewed changes
gitless/cli/gl_switch.py Outdated
if not b:
pprint.err('Branch {0} doesn\'t exist'.format(args.branch))
pprint.err_exp('to list existing branches do gl branch')
pprint.err_exp('to create a new branch do gl branch -c feature/foo')
Copy link
Member

@spderosso spderosso Feb 27, 2017

Choose a reason for hiding this comment

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

You could pass in the name of the branch the user provided:

pprint.err_exp('to create a new branch do gl branch -c {0}'.format(args.branch))

Copy link
Contributor Author

@katrinleinweber katrinleinweber Feb 27, 2017

Choose a reason for hiding this comment

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

Thanks for the hint :-)

@spderosso spderosso merged commit 6cde86e into gitless-vcs:master Feb 27, 2017
@spderosso
Copy link
Member

spderosso commented Feb 27, 2017

thank you!

@katrinleinweber
Copy link
Contributor Author

katrinleinweber commented Feb 28, 2017

You're welcome :-) I thank you for working on gitless & researching domain-specific Gits!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@spderosso spderosso spderosso left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Make switch command a bit more helpful

2 participants

@katrinleinweber @spderosso