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

Create chapter key as guide for learners and developers #27

Merged
merged 2 commits into from
Sep 9, 2024

Conversation

joelostblom
Copy link
Owner

We can add to this as more items come up, but I think this captures what we included in chapter 1

Copy link
Collaborator

@rorywhite200 rorywhite200 left a comment

Choose a reason for hiding this comment

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

This seems like a good intro. I note that Quarto Live has its own format for hints and solutions so we can add that later if we are using it.

Screenshot 2024-09-09 at 9 48 02 AM

Copy link

github-actions bot commented Sep 9, 2024

Hello! I've built a preview of your PR so that you can compare it to the current main branch:

@joelostblom
Copy link
Owner Author

Yes, hopefully we can use the quarto live hints and solutions for some of the exercises. Part of me wants to make it so that every single code box is editable (or at least that there is a button to "edit this code"), so that students can play around with anything to deepen there understanding. I don't want to spend more time on that formatting now though, so we're tracking it separately in #16

As you can see above we now also have the actions preview workflow up and running, so it will be easy to assess the visual changes introduced by a PR. Note that it seems to take a few seconds for it to be available after the comment is posted

@joelostblom joelostblom merged commit dbc97f5 into main Sep 9, 2024
1 check passed
@joelostblom joelostblom deleted the chapter-key branch September 9, 2024 23:11
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