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

updates to the CLI chapter #52

Merged
merged 20 commits into from
Oct 26, 2023
Merged

updates to the CLI chapter #52

merged 20 commits into from
Oct 26, 2023

Conversation

lnnrtwttkhn
Copy link
Owner

to prepare for session 2 of this course, we make updates mainly to the CLI chapters.

fixes #51

@lnnrtwttkhn lnnrtwttkhn mentioned this pull request Oct 23, 2023
28 tasks
@konradpa
Copy link
Contributor

was able to add/fix most of the things on the list.
some notes:

  • i wasnt able to render the page at first, had to delete a line in the settings quarto.yml and in contents.qmd to do it. Maybe the fault of remote setup so I didnt commit my changes.

  • Is the copyright for memes important? I think its hard to find memes with CC BY licence

  • is it really necessary to switch to bash shell if zsh is installed? I think there a no differences to the commands we use.

@lnnrtwttkhn
Copy link
Owner Author

thank you for all these changes @konradpa, looks great!

i wasnt able to render the page at first, had to delete a line in the settings quarto.yml and in contents.qmd to do it. Maybe the fault of remote setup so I didnt commit my changes

hmm, no idea what this could be about.
did you get any error message?

Is the copyright for memes important? I think its hard to find memes with CC BY licence

haha, yes, probably no licenses for memes. should be fine.

is it really necessary to switch to bash shell if zsh is installed? I think there a no differences to the commands we use.

you are right. but good to be aware of the difference. the way that this is covered in the book now is fine for our purposes.

@lnnrtwttkhn lnnrtwttkhn merged commit 39e1e40 into main Oct 26, 2023
1 check passed
@lnnrtwttkhn lnnrtwttkhn deleted the cli branch October 26, 2023 13:37
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.

preparation of cli chapter
2 participants