-
Notifications
You must be signed in to change notification settings - Fork 6.1k
Add LLL chapter in the documentation #5411
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fine, only minor comments.
Codecov Report
@@ Coverage Diff @@
## develop #5411 +/- ##
=========================================
Coverage ? 88.1%
=========================================
Files ? 323
Lines ? 32647
Branches ? 3873
=========================================
Hits ? 28763
Misses ? 2585
Partials ? 1299
|
docs/lll.rst
Outdated
|
||
.. _lll: | ||
|
||
LLL is a low-level language for the EVM based on s-expressions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe: with an s-expressions syntax.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It has like 4-5 acronyms, one other: Little Lovely Language
.
I'd avoid it in this PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the s-expressions
part clarifies that it is "lisp like", but feel free to open a PR with changes to the LLL chapter.
@@ -88,3 +88,4 @@ Contents | |||
bugs.rst | |||
contributing.rst | |||
frequently-asked-questions.rst | |||
lll.rst |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we really want to add another high level entry to the toc?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we really want to add another high level entry to the toc?
Yes, that is the easy way to find it. I hope we remove all of LLL with 0.6.0.
Closes #5393.