Skip to content

Conversation

@chlin501
Copy link

@chlin501 chlin501 commented Nov 2, 2025

  • Update the author information to src/data/authors directory

  • Add the article build your own async to src/data/articles directory

@chlin501
Copy link
Author

chlin501 commented Nov 2, 2025

Hi @daniel-ciocirlan, I follow the CONTRIBUTING.md doc. But please let me know if anything that needs to be updated/ corrected at my side. Many thanks.

Copy link
Collaborator

@daniel-ciocirlan daniel-ciocirlan left a comment

Choose a reason for hiding this comment

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

I think this is a valuable guide, with lots of lessons inside. I mostly commented on the flow of the article and some English nits.

The big thing that's missing in this article is (maybe surprisingly) "async", which we usually understand as the ability to run computations on callbacks later. The current code should allow this extension, and it would be valuable to add. If you don't want to add callbacks and just focus on coroutines and event loops, we can describe the article as such, and cut "async" out.

I'm very eager to read this article again after addressing the comments. Thank you for it.

@chlin501 chlin501 changed the title Build your own async Coroutines, Event Loops - Build Your Own in Scala Nov 11, 2025
@chlin501
Copy link
Author

Hi @daniel-ciocirlan, I updated the article and related code based on the comments. Please let me know if any issues. Many thanks.

@chlin501
Copy link
Author

chlin501 commented Nov 11, 2025

extension

I may revisit this later - allowing to run computations on callbacks later. At the moment, I just cut out the async part. Thanks.

Edit:
In addition, I also updated this PR name to reflect the title of this article.

Copy link
Collaborator

@daniel-ciocirlan daniel-ciocirlan left a comment

Choose a reason for hiding this comment

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

One last comment about the photo (up to you to decide), otherwise the article looks good.

Will merge after the Black Friday sale so that your article can have the visibility it deserves.

@chlin501
Copy link
Author

I am perfectly fine with my face shown up blurry on the blog. Thank you again for your advice. and efforts. I sincerely appreciate it!

One last comment about the photo (up to you to decide), otherwise the article looks good.

Will merge after the Black Friday sale so that your article can have the visibility it deserves.

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