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

Soundcloud #50

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Soundcloud #50

wants to merge 8 commits into from

Conversation

thanithani
Copy link
Collaborator

It works, including previews.

Should consider rewriting using the soundcloud gem: https://github.com/soundcloud/ruby-api-wrapper/wiki

@@ -250,7 +250,7 @@ GEM
temple (~> 0.4.1)
tilt (~> 1.3.3)
slop (2.4.4)
springboard (0.18.7.1)
springboard (0.19.9)
Copy link
Owner

Choose a reason for hiding this comment

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

Get rid of the changes in this file: git checkout master Gemfile.lock

@arbales
Copy link
Collaborator

arbales commented Oct 11, 2012

Some Git tips…

You can avoid accidental additions to files by using git add -p instead of adding file-by-file. It'll step you through each potential change and as if you want to add them, discard, them or split them into smaller chunks. It's also a way to ensure that you do self code review.

Also, if you find you have several changes in a row that are ostensibly the same thing on a branch, like removing different accidentally added files, you can use git commit --amend to add to your last commit. If you haven't pushed to GitHub yet, there's no caveats. If you have, you'll have to be careful while force pushing only your branch: git push -f origin soundcloud.

@thanithani
Copy link
Collaborator Author

Oh add -p, killer. That will definitely be handy.

Thanks for all the help you two and Archit have given this week. Its been really fun, actually. Excited to get my hands dirtier...

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

3 participants