Skip to content

Conversation

@BitPupper
Copy link
Contributor

Nono Horiuchi

Too Long

  • Chapter 6
  • 6.5 Iterating Through a List

Integer Info

  • Chapter 6
  • 6.1 Intro to Lists

Snookle Game

  • Chapter 6
  • 6.5 Iterating Through a List

Monty Hall

  • Chapter 6
  • 6.4 Manipulating Lists

Virtual Pet

  • Chapter 5
  • 5.4 For Loops vs. While Loops

Although it's fixed now, the chapter 6 practice problems did not originally have practice and solutions folders, so I put them all into the same directory as the examples folder, and named the files to distinguish between problem and solution. Sorry for the inconvenience!

@phrdang phrdang self-requested a review July 1, 2020 03:37
@phrdang
Copy link
Member

phrdang commented Jul 1, 2020

Hey Nono, firstly, you're a rockstar! Thank you for continuing to contribute all of these practice problems!

Secondly, on the issue with the folders not appearing on GitHub, you can actually push any folders you make on your local repository to GitHub, so you can actually just make folders called "practice" and "solutions" and put them there in the future.

Thirdly, before pushing your local branch, make sure that you have pulled changes from remote to avoid merge conflicts. For example, in this PR (pull request) you also pushed your solution code to the Temperature problem you submitted a while back. However, in the time since that happened, I had already updated the solution code. Basically, your code was a bit outdated, which results in a conflict. To avoid this, when you think you're ready to push your committed changes, make sure to pull to get the latest code. (In fact, you'll probably want to do this often as you work on problems to make sure you have the most recent info.)

If you have any questions about using git or GitHub, please let me know on Discord and I'd be happy to help! In the meantime, I'll be looking at your code and submitting a review soon :)

phrdang added a commit that referenced this pull request Jul 1, 2020
@phrdang phrdang merged commit 66d637f into code4tomorrow:master Jul 1, 2020
@phrdang phrdang removed their request for review July 1, 2020 20:03
@BitPupper
Copy link
Contributor Author

Hi, thanks, that really helped! I'll make sure to do that in the future.

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