Skip to content

Conversation

@WoobieTheBear
Copy link

that’s how you can change someone else' code and suggest changes:

  • you go to the repository
  • you open the code you want to change
  • you click edit
  • you change the code
  • you click preview
  • you write a short explanation what you did
  • you click propose File change

And all the other person has to do now is to accept the pull request :-D i just figured that out myself =)

that’s how you can change someone else' code and suggest changes:
- you go to the repository
- you open the code you want to change
- you click edit
- you change the code
- you click preview
- you write a short explanation what you did
- you click propose File change

And all the other person has to do now is to accept the pull request :-D i just figured that out myself =)
@DigiWongaDude
Copy link
Owner

Great! Thanks for the really useful tips!

I've also made a fully functioning (though not finished) Ms Pacman, so I
have a code template in my mind now for doing collision detection (think
about this: you can't put listeners on every dot, or use pac's listener to
check every dot!), layered sound mapping, monster A.I. and layered keyboard
controls (in Pacman you have to buffer a keyboard 'choice' otherwise the
player can only turn a corner AT that corner. I've seen some others get
stuck with that bit, and the gameplay goes to zero (terrible).

Anyway, thanks again (I won't bombard you with links or emails today (or
usually)), I'm just excited to be chatting to someone who isn't a complete
dickhead and knows all about this stuff!. lol.

Cheers

Digi

On 9 December 2013 13:26, Lukas notifications@github.com wrote:

Reopened #1 #1.


Reply to this email directly or view it on GitHubhttps://github.com//pull/1
.

@DigiWongaDude
Copy link
Owner

Oh wow, I see my reply came here. That's cool, I edited out the image parts. :-) Nice!

@WoobieTheBear
Copy link
Author

yes the web is 99% full of trolls and smartypants =D
cheers

Copy link
Owner

Choose a reason for hiding this comment

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

Note: I see you deleted this line.

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