Skip to content

Conversation

@LomaxOnTheRun
Copy link
Collaborator

Yatzy and FizzBuzz katas for the 24-05-2018 coding dojo session

@carvallegro
Copy link
Contributor

@LomaxOnTheRun I believe there is a bit of confusion here between. With the new guidelines we are dropping the folder dating system (ie 2018/24may...) instead we just do the exercise in the kata folder and push the result.

Put the 2018/24-05-2018/fizz-buzz/index.js file back in katas/fizz-buzz.

Also, how come you have modification in the yatzy folder?

@LomaxOnTheRun
Copy link
Collaborator Author

LomaxOnTheRun commented May 29, 2018 via email

@LomaxOnTheRun
Copy link
Collaborator Author

Urgh, alright, I've tried changing the branch via terminal commands but for some reason, I don't seem to have permission to do that. To get the files up in the first place I added them manually on Github (which it seems fine to let me do), just no terminal commands. If you can get rid of the last two commits on here, I can create another pull request with my stuff. Sorry about all the hassle :/

@LomaxOnTheRun LomaxOnTheRun force-pushed the melbourne/24-05-2018 branch from 8c1e411 to de208b3 Compare May 31, 2018 12:06
@LomaxOnTheRun
Copy link
Collaborator Author

Alright, I've sorted out my permissions issue, and I've pushed the files to (hopefully) the right place now. Let me know if it's right, otherwise I can change it easily enough now :)

@carvallegro carvallegro merged commit e8be523 into master Jun 1, 2018
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.

3 participants