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

Working copy #23

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

Conversation

ChantalDemissie
Copy link

Adagrams

Congratulations! You're submitting your assignment.

Comprehension Questions

Feature Feedback
What are the components that make up a method? Method signature, parameters(if any) block, return value.
What are the advantages of using git when collaboratively working on one code base? Easily view and edit file, without the other person (not having to email copies to each other). It tracks all changes and keeps a log of them (when comitted).
What kind of relationship did you and your pair have with the unit tests? A tumultous one. We learned how to read unit testing a lot better and actually understand how it works and apply that to what kind of code we need to write. Helpful for debugging as well.
Does your code use any methods from the Enumerable mixin? If so, where and why was it helpful? we used a map method to return the word/score for finding out what word won.
What was one method you and your pair used to debug code? Pry, printing to the screen, unit tests were helpful.
What are two discussion points that you and your pair discussed when giving/receiving feedback from each other that you would be willing to share? We discussed out learning styles and execution styles with our coding techniques and recognized it would be great to go over that from the start and come up with a plan of action to use eachother's strengths before tackling individual problems in the project.

@tatsqui
Copy link

tatsqui commented Feb 23, 2019

We didn't realize that pushing from what we named our branch would be the name of our submission! Whoops! - Tatiana and Chantal

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