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

Blog post: Using Expressions Effectively in Rust #24

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

mre
Copy link
Contributor

@mre mre commented May 15, 2024

Wrote an article on expressions in Rust and why I think they are great.
This is version 2, which is an attempt to capture some of the magic of expressions.

@mre mre requested a review from M3t0r May 15, 2024 23:27
content/blog/expressions/index.md Outdated Show resolved Hide resolved
content/blog/expressions/index.md Outdated Show resolved Hide resolved
content/blog/expressions/index.md Outdated Show resolved Hide resolved
content/blog/expressions/index.md Outdated Show resolved Hide resolved
content/blog/expressions/index.md Outdated Show resolved Hide resolved
@mre mre force-pushed the blog/expressions branch 3 times, most recently from ef37144 to ff99b29 Compare August 1, 2024 15:36
@mre
Copy link
Contributor Author

mre commented Aug 1, 2024

This article is now ready for another round of reviews. I completely rewrote it from scratch to focus more on what I really feel when I write expressions in Rust, not just purely on the mechanics of it all. If someone has time for a review, I'd appreciate the feedback.

content/blog/expressions/index.md Outdated Show resolved Hide resolved
content/blog/expressions/index.md Outdated Show resolved Hide resolved
content/blog/expressions/index.md Outdated Show resolved Hide resolved
content/blog/expressions/index.md Outdated Show resolved Hide resolved
content/blog/expressions/index.md Outdated Show resolved Hide resolved
content/blog/expressions/index.md Outdated Show resolved Hide resolved
content/blog/expressions/index.md Outdated Show resolved Hide resolved
content/blog/expressions/index.md Outdated Show resolved Hide resolved
content/blog/expressions/index.md Outdated Show resolved Hide resolved
@mre
Copy link
Contributor Author

mre commented Aug 7, 2024

@M3t0r, @thomas-zahner I updated the post according to your feedback. Hopefully, I also replied to all your nice comments. Thanks for the thorough feedback!
If you don't mind, please take another look. 😊

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