Skip to content

Tutorial pull request#6

Closed
snoeg wants to merge 3 commits into
sparkfun:masterfrom
snoeg:master
Closed

Tutorial pull request#6
snoeg wants to merge 3 commits into
sparkfun:masterfrom
snoeg:master

Conversation

@snoeg

@snoeg snoeg commented Aug 9, 2013

Copy link
Copy Markdown

Thanks for the tutorial. However, I wasn't able to commit the changes into the local repository using GitHub for Windows. Everything else worked fine. I always got an error, but the dialogue box let me open powershell and from there the command "git commit" worked. (I'm running WinXP and have admin privileges.) Anyway, thanks for providing this tutorial. It helps a lot as I'm just getting started with git/version control. :)

@ToniCorinne

Copy link
Copy Markdown

Thanks for submitting the request! I'm not going to pull in your fixes (so that way the repo keeps the bugs for others working through the tutorial), but it looks like you got everything working! Out of curiosity, what error were you getting through the GUI?

@ToniCorinne ToniCorinne closed this Aug 9, 2013
@snoeg

snoeg commented Aug 9, 2013

Copy link
Copy Markdown
Author

That reply was fast :) I'm starting to see that Git is quite useful. The error was a vague "Failed to create new commit". I quickly made a screenshot: https://dl.dropboxusercontent.com/u/44724561/gitError.jpg
Anyway, I'll try again on a different machine later. Thanks for replying.

@ToniCorinne

Copy link
Copy Markdown

Yea, no worries. That is an odd bug we keep running into here as well. I think it's some bug in the GUI, but we haven't really been able to track it down yet. Personally, I love the command line, but I know a lot of people around here find the GUI fast and slick. Best of luck as you keep moving forward with this!

@snoeg

snoeg commented Aug 19, 2013

Copy link
Copy Markdown
Author

Thanks for the wishes. I've now managed to fix the "Failed to create new commit" problem by entering my email address in the options window. (From the main window, "tools" -> "options". I didn't put an address there before, because of the warning that it would be public.) I just thought I'd let you know in case someone else has the same problem. Have a nice week :)

@ToniCorinne

Copy link
Copy Markdown

Thanks for the update! I'm sure that information will come in handy for others down the road. Thanks for sharing the info :)

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.

2 participants