Skip to content

Conversation

@liamboone
Copy link

I wasn't quite sure what was meant by "check in," but I've committed to my fork and here's a pull request. Quite new to git/github

Liam

@betajippity
Copy link
Member

Pull request acknowledged!

(My bad, I said earlier that we didn't want pull requests to the master, but we actually do.)

On Tuesday, September 18, 2012 at 9:11 PM, liamboone wrote:

I wasn't quite sure what was meant by "check in," but I've committed to my fork and here's a pull request. Quite new to git/github
Liam
You can merge this Pull Request by running:
git pull https://github.com/liamboone/Homework0 master
Or view, comment on, or merge it at:
https://github.com/CIS565-Fall-2012/Homework0/pull/1
Commit Summary
Added Screenshot as directed, also had to use the glut32.lib fix sugg…
ACTUALLY added screenshot...

File Changes
M HW0_WIN/CUDATEST/main.cpp (5)
A HW0_WIN/Debug/CUDATEST.exe (0)
A HW0_WIN/Debug/CUDATEST.ilk (0)
A HW0_WIN/Debug/CUDATEST.pdb (0)
A HW0_WIN/Debug/freeglut.dll (0)
A HW0_WIN/Debug/glut32.dll (0)
A HW0_WIN/shared/freeglut/glut32.dll (0)
A HW0_WIN/shared/freeglut/lib/glut32.lib (0)
A proof.png (0)

Patch Links
https://github.com/CIS565-Fall-2012/Homework0/pull/1.patch
https://github.com/CIS565-Fall-2012/Homework0/pull/1.diff


Reply to this email directly or view it on GitHub (https://github.com/CIS565-Fall-2012/Homework0/pull/1).

@betajippity betajippity reopened this Sep 19, 2012
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