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

Interactive hyperbole demo #292

Merged
merged 10 commits into from
Oct 31, 2023
Merged

Conversation

chookity-pokk
Copy link
Contributor

@chookity-pokk chookity-pokk commented Dec 31, 2022

Hank here, we communicated over the summer via email about my work on these documents.

Sorry this has taken forever, I am finally on a break from grad school. Here though is the interactive hyperbole demo I have been working on. I figured we didn't want to replace the current demos with these interactive ones I've been working on, so I made a new directory for them. I also have a hyrolo one that I am virtually done with and a koutliner one that is very close to being done.

I am very open to suggestions on changing names or anything like that!

@chookity-pokk
Copy link
Contributor Author

Also, I figured it'd be better to create a PR for each document so it's easier to review and get merged in? If not, I can add them all into this PR.

@rswgnu
Copy link
Owner

rswgnu commented Dec 31, 2022 via email

interactive-demos/HYPERBOLE Outdated Show resolved Hide resolved
interactive-demos/HYPERBOLE Outdated Show resolved Hide resolved
interactive-demos/HYPERBOLE Outdated Show resolved Hide resolved
interactive-demos/HYPERBOLE Outdated Show resolved Hide resolved
interactive-demos/HYPERBOLE Outdated Show resolved Hide resolved
Copy link
Collaborator

@matsl matsl left a comment

Choose a reason for hiding this comment

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

Good work. Left some comments you might want to address.

chookity-pokk and others added 2 commits January 1, 2023 16:59
Co-authored-by: Mats Lidell <mats.lidell@lidells.se>
@chookity-pokk
Copy link
Contributor Author

I added in the recommended changes!

@matsl matsl requested a review from rswgnu January 1, 2023 23:33
Copy link
Collaborator

@matsl matsl left a comment

Choose a reason for hiding this comment

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

LGTM

@chookity-pokk
Copy link
Contributor Author

Just a gentle ping here.

@chookity-pokk
Copy link
Contributor Author

@rswgnu I've been crazy busy the last couple months with finishing grad school and moving across the country, but now I finally have more time to wrap all the tutorials up. So out of curiosity, what am I missing from this to get it merged? No rush, of course.

@rswgnu rswgnu merged commit 520f47c into rswgnu:master Oct 31, 2023
5 checks passed
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.

None yet

3 participants