WIP: Add more documentation to trustfall_core#136
WIP: Add more documentation to trustfall_core#136ginger51011 wants to merge 7 commits intoobi1kenobi:mainfrom
trustfall_core#136Conversation
|
This is awesome, thank you so much for taking the initiative on this 🎉 I'm currently exhausted from staying up half the night answering questions on HackerNews, but I will read and review ASAP. For easier merging, consider splitting up the docs so that the additions to each file are its own pull request? That way each file can be documented, discussed, and merged separately from all the other files. In my experience, this will make us able to move a lot faster. |
Thanks! No stress at all, it's pretty crazy seeing the repo going from ~500 stars to 1,1k in like a day! This could probably get merged now (if the quality is up to par), and I could organize further PRs in a more organized way. This is pretty much me trying to understand the code by writing notes here and there |
Apparently Trustfall was the #1 most-starred project on GitHub yesterday: https://twitter.com/PredragGruevski/status/1623711641127747584 Absolutely wild! 🚀
Based on a quick glance I think some small edits for clarity would be in order, largely due to my poor choices of names that then require more documentation than otherwise would have been required. Let's split the difference: would you mind splitting this into PR-per-directory, and using PR-per-file for the rest? You probably know this already, but just in case: |
15f64a4 to
64ebfd5
Compare
This is a very WIP project related to #135.
Right now there isn't much documentation at all, so I thought I might help get the
rustdocat least a bit more easy to navigate. It's a bit write-as-I-go, but I'm open to feedback and suggestions from anyone!