Skip to content

Conversation

@MarayahAy
Copy link
Collaborator

No description provided.

@fdabl fdabl requested review from AlexisDaniels and petersen-f May 19, 2021 11:56
print ('okay let me check what recipes we have available with', ingredients, '...')


#importing recipes and store in dict varibale

Choose a reason for hiding this comment

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

typo "variable"



#importing recipes and store in dict varibale
import csv

Choose a reason for hiding this comment

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

for style purposes, normally, people like to import all packages at the start of the script. But it is just a styling suggestion and style is subjective (maybe). :P

Copy link

@AlexisDaniels AlexisDaniels left a comment

Choose a reason for hiding this comment

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

Not much to say about it - the idea looks cool.

I just made a couple of style/typo comments

Copy link

@AlexisDaniels AlexisDaniels left a comment

Choose a reason for hiding this comment

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

I guess you were just testing the unit tests here? I mean, it has no correspondence to your real code, right?

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