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 link shortening option #21

Merged
merged 3 commits into from Nov 11, 2012
Merged

add link shortening option #21

merged 3 commits into from Nov 11, 2012

Conversation

dlo
Copy link
Contributor

@dlo dlo commented Nov 10, 2012

This commit gives jist the new ability to shorten gist URLs using GitHub's URL shortener, git.io. It adds two options on the command line (-s and --shorten).

If the shortening step fails, or if git.io is down, jist will just return the normal, unshortened version.

This commit gives jist the new ability to shorten gist URLs using
GitHub's URL shortener, git.io. It adds two options on the command
line (`-s` and `--shorten`).

If the shortening step fails, or if git.io is down, jist will just
return the normal, unshortened version.
@rking
Copy link
Collaborator

rking commented Nov 10, 2012

Coolness. Can you throw in some tests?

I'd be happy with either mocks or live hits to the URL, whichever you want to do.

Might require a bit of refactoring to be easy to test.

@dlo
Copy link
Contributor Author

dlo commented Nov 10, 2012

I don't normally do Ruby stuff, so let me know if the tests look ok. I also updated the Gemfile--I needed to install the "extensions" gem, otherwise the tests wouldn't run for me.

@ConradIrwin ConradIrwin merged commit 9bf41b7 into ConradIrwin:master Nov 11, 2012
@ConradIrwin
Copy link
Owner

Awesome, thanks for this!

@rking
Copy link
Collaborator

rking commented Nov 13, 2012

Yep, really nice feature.

@mattn — can you confirm that this works on your proxy setup?

@mattn
Copy link
Contributor

mattn commented Nov 13, 2012

It need bits fix to post to git.io 8e64022

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.

None yet

4 participants