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

Initial Contribution #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

theredwillow
Copy link

Changes:

  1. Minor PEP8 fixes
  2. Added text file saving and reading functionality for later use and
    printing needs
  3. Added command variable to hold new_stuff().upper() to handle case
    incompliance in user input
  4. Added immediate REMOVE functionality (i.e. "Remove 4" to remove
    shopping_list[3] without additional dialog, including via strings such
    as "Remove potatoes" to remove item from shopping_list equal to
    "potatoes" and error handling for "This isn't in your list.")
  5. show_list() now called before asking user which item to remove
  6. show_list() now has return_only variable: true to return shopping
    list, default false to print shopping list - useful for printing inside
    functions such as input() and write()
  7. for loop in show_list() now enumerates to eliminate use of local
    count variable

New requirements:
import os.path.isfile

Changes:
1. Minor PEP8 fixes
2. Added text file saving and reading functionality for later use and
printing needs
4. Added command variable to hold new_stuff().upper() to handle case
incompliance in user input
3. Added immediate REMOVE functionality (i.e. "Remove 4" to remove
shopping_list[3] without additional dialog, including via strings such
as "Remove potatoes" to remove item from shopping_list equal to
"potatoes" and error handling for "This isn't in your list.")
4. show_list() now called before asking user which item to remove
5. show_list() now has return_only variable: true to return shopping
list, default false to print shopping list - useful for printing inside
functions such as input() and write()
6. for loop in show_list() now enumerates to eliminate use of local
count variable

New requirements:
import os.path.isfile
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.

1 participant