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

doc: misc improvements - WIP #1273

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nielsbom
Copy link
Contributor

Not done yet, but I thought I'd share because "shitty first draft"

Some thoughts already:

I added some content for first users in usage.md just to get them off the ground.

I think the emphasis on "functions" in the docs is a bit strange. But I can write code so I'm maybe not representative. I think it's better to say: commands contain code and then talk about what kinds of code you can write and what (modules, properties, methods) you can use.

More distinction between commands and command utilities is useful I think.

I think it's OK to be explicit with: "the code between the tags of a command is JavaScript code".

I think the name "JavaScript execution commands" is a bit awkward because interpolation commands also contain JavaScript and also execute. But that naming decision can't easily be undone. I do prefer "execution commands" over "JavaScript execution commands" because it's shorter. It also mirrors "interpolation commands" wrt name length and thus contrasts better with it.

@nielsbom nielsbom changed the title doc improvements - WIP doc: misc improvements - WIP Dec 22, 2023
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.

1 participant