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

Exercise 5 Update #8

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

Conversation

TheBigH50
Copy link

Good to go!

Copy link

@hunterandersen hunterandersen left a comment

Choose a reason for hiding this comment

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

While I do love the fun bacon-based ipsum, I will request it gets changed to either the basic lorem ipsum, or a different ipsum. Something about it just doesn't feel professional enough and we don't want to send the wrong message, particularly so early in the course.

@TheBigH50
Copy link
Author

Removed baconIpsum!

Copy link

@hunterandersen hunterandersen left a comment

Choose a reason for hiding this comment

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

Yeah, the regular lorem ipsum does come across more professionally

@hunterandersen
Copy link

I think we'll need to check with Jeremy, John, or Amoriss about the changes to make sure it doesn't blindside them. I think this is a shared repo for all TrueCoders classes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants