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

Black pepper tofu fixes #49

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

Black pepper tofu fixes #49

wants to merge 3 commits into from

Conversation

matthewdking
Copy link
Member

@matthewdking matthewdking commented Nov 6, 2017

relates #48

Changed the colour of cost badge and removes unnecessary title text.

Double check the rest of the recipe again to check other mistakes didn't slip through please ❤️

relates #48
Copy link
Contributor

@mattlub mattlub left a comment

Choose a reason for hiding this comment

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

How many does this recipe serve?

added a comma cos cba reviews
Copy link
Contributor

@m4v15 m4v15 left a comment

Choose a reason for hiding this comment

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

I would maybe move one of the images to the top (the first one?)

@matthewdking
Copy link
Member Author

@m4v15 I thought this would be against the contributing guide, no? 😕

@m4v15
Copy link
Contributor

m4v15 commented Nov 6, 2017 via email

Copy link
Contributor

@finnhodgkin finnhodgkin left a comment

Choose a reason for hiding this comment

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

The PR looks good to me, but I spotted a minor typo elsewhere in the recipe: the 'full' on this line should be 'fully'.

@Karyum
Copy link
Contributor

Karyum commented Jan 5, 2018

@matthewdking I'm happy to merge as soon as Lewbes's requested changed is resolved

@mattlub
Copy link
Contributor

mattlub commented Jan 6, 2018

@Karyum but you would wait for me to approve the changes of course?

@Karyum
Copy link
Contributor

Karyum commented Jan 6, 2018

@mattlub nope

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.

5 participants