Skip to content

Made text edits on the language of the document. #9

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

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

Conversation

churcho
Copy link

@churcho churcho commented Nov 27, 2018

Made minor edits in the language of the text.

In week 6, do you think it would be a good option to include a link to credo? I find it useful to introduce code analysis at an earlier stage to allow one to get proper foundation on working standards in teams and conforming to these, what are your opinions?

@zacck-zz
Copy link
Member

zacck-zz commented Nov 27, 2018

@churcho yeah you can add a credo lesson perhaps you can consider adding usage of git hooks in combination with credo, see this link
https://github.com/LittleKidogo/spender/blob/develop/docs/setup.md and the same project too for a setup that uses githooks and credo

Perhaps you can distill this down it to a lesson and a challenge? Let me know if I can give you a hand at anything

Add credo to list of libraries to look up
@churcho
Copy link
Author

churcho commented Nov 27, 2018

Added credo as a topic to cover in week 6. Will submit a lesson and challenge PR for this as well.

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