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

Demo Day November Re-cap #153

Merged
merged 18 commits into from
Feb 7, 2023
Merged

Conversation

avriiil
Copy link
Contributor

@avriiil avriiil commented Nov 21, 2022

No description provided.

@mrocklin
Copy link
Member

@jacobtomlinson do you have time to review?

Copy link
Member

@jacobtomlinson jacobtomlinson left a comment

Choose a reason for hiding this comment

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

This is awesome, it's exactly the kind of digest from the demo day that I had in mind. Thanks for taking the time to write it.

I've made a bunch of comments, mostly I think we should just be linking all of the things. These kinds of digests should just be a giant pile of links to other things.

.gitignore Show resolved Hide resolved
_posts/2022-11-21-november-demo-day.md Outdated Show resolved Hide resolved
_posts/2022-11-21-november-demo-day.md Outdated Show resolved Hide resolved
_posts/2022-11-21-november-demo-day.md Outdated Show resolved Hide resolved
_posts/2022-11-21-november-demo-day.md Outdated Show resolved Hide resolved
_posts/2022-11-21-november-demo-day.md Outdated Show resolved Hide resolved
_posts/2022-11-21-november-demo-day.md Outdated Show resolved Hide resolved
_posts/2022-11-21-november-demo-day.md Outdated Show resolved Hide resolved
_posts/2022-11-21-november-demo-day.md Outdated Show resolved Hide resolved
@jacobtomlinson
Copy link
Member

Also looks like the post failed linting. You might want to set up pre-commit.

@GenevieveBuckley
Copy link
Collaborator

I notice we haven't pushed this draft over the finish line. It might be perfect timing if we published this before February's demo day (assuming that's scheduled, I might ask over at dask/community#296 to double check)

I don't think it would take much more to get it up (running the linter and adding some of the links Jacob suggested). If @rrpelgrim is busy and would like it, I could probably do those tasks.

@avriiil
Copy link
Contributor Author

avriiil commented Feb 2, 2023

Thanks @GenevieveBuckley -- I'm not able to wrap this up at the moment but am happy for others to put on the final touches.

@GenevieveBuckley GenevieveBuckley merged commit d4dcfb7 into dask:gh-pages Feb 7, 2023
@GenevieveBuckley
Copy link
Collaborator

A big thank you @rrpelgrim for writing this up! (And thank you Jacob for reviewing it)

I have added links everywhere, fixed the linting, fixed the screenshot image display problem, and checked how it looks when I serve it with jekyll locally. We should be good to go 🚀

@GenevieveBuckley
Copy link
Collaborator

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.

4 participants