-
Notifications
You must be signed in to change notification settings - Fork 10
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
Improve the DSL docs page #146
Conversation
@@ -1,4 +1,4 @@ | |||
include .special.rst | |||
.. include:: .special.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.
This is literally at the top of the page. Yikes
https://nuclear.readthedocs.io/en/latest/dsl.html
<font color="blue">DSL Request</font><br /> | ||
|
||
This is :red:`red !` And :blue:`this part is blue`. | ||
:blue:`On<Dsl, Word(s)>`\(:red:`Runtime, Args`).then(:green:`CallbackFunction`); |
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.
If we want to use rst
, we should actually use it.
Also, that escaped paren is deliberate. It's required after the backtick
Finally, the developer can define the callback which will execute when the reaction is triggered during runtime. The | ||
callback can be defined using a C++ lambda function. | ||
callback can be defined using a C++ lambda expression. |
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.
ACKSHUALLY 🤓
these emissions can take place: | ||
|
||
.. todo:: | ||
|
||
Trent - I need to decide and get consistent on what we will call the powerPlant. Should it be PowerPlant or |
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.
Making an executive decision here
Mostly pretty easy stuff. Opted for
rst
features instead of HTML as well.