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

Already on GitHub? Sign in to your account

add option to print the url, allowing jumping to stories directly from the command line #7

Merged
merged 7 commits into from Dec 11, 2011

Conversation

Projects
None yet
3 participants
Contributor

eschulte commented Dec 10, 2011

I never use python, so take these changes with a grain of salt

Owner

pkrumins commented Dec 11, 2011

why did you remove beautifulsoup?

it's there because other versions are broken, so i ship hacker-top with the working one. ... but that was actually like 3 years ago when i wrote it, perhaps things have changed, so i'll just merge this pull.

pkrumins added a commit that referenced this pull request Dec 11, 2011

Merge pull request #7 from eschulte/master
add option to print the url, allowing jumping to stories directly from the command line

@pkrumins pkrumins merged commit ea99ac9 into pkrumins:master Dec 11, 2011

Contributor

eschulte commented Dec 11, 2011

I was working from the git version installed from the Arch linux user repository (AUR), so my pull request accidentally included the commits included in that repository. So basically I don't know why beautifulsoup was removed, but it sounds like things should work moving forward.

Thanks for sharing hacker/reddit_top, I'm enjoying both -- Eric

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