-
-
Notifications
You must be signed in to change notification settings - Fork 51
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
SDC Tools Module Sprint 2 #1227
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for cyf-curriculum ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for cyf-programming ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for cyf-launch ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for cyf-common ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for cyf-piscine ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for cyf-sdc ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
25ed5ae
to
c0b34be
Compare
✅ Deploy Preview for cyf-itd canceled.
|
✅ Deploy Preview for cyf-tracks ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for cyf-itd ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for cyf-tracks ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this was in pending sorry, tiny tweaks
common-content/en/module/tools/programming-language-concepts/index.md
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some minor comments & suggestions, but looks pretty good!
5="TODO: Pipe the output of jq to something interesting" | ||
1="Describe what a shell pipeline is." | ||
2="Explain why we use shell pipelines." | ||
3="Explain the difference between stdout and stderr." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a missing LO about stdin?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's an interesting question...
I'm not actually sure understanding stdin is super important, other than being able to construct a shell pipeline (where a very hand-waving understanding is probably sufficient).
I'm not sure we're ever going to have our trainees write programs which reads from stdin, and honestly in my real professional life I basically ever do. WDYT?
477da20
to
dc98c37
Compare
Co-authored-by: Sally McGrath <sally@codeyourfuture.io>
Co-authored-by: Sally McGrath <sally@codeyourfuture.io>
fea339a
to
a5ff523
Compare
What does this change?
Adds SDC Tools Module Sprint 2 covering:
jq
Common Content?
Yes - adds a bunch of blocks
Org Content?
Yes, sets up a sprint