This repository has been archived by the owner on Oct 16, 2024. It is now read-only.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Semi-autogenerated cli reference #172
Semi-autogenerated cli reference #172
Changes from 4 commits
4840479
2fa1e07
7b83973
d48ed0d
96f2f7b
a7085ab
de0b20a
65f1588
e0c1b9d
7182940
5b221ba
2ed3efa
e196fc4
fabe888
e669838
f254cb6
095acea
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
MODEL -> model
required / optional check how it's done in other docs
list with options - it was better the way it was done (compact, easier to read, etc)
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.
required and optional are different things: required means you always should provide it, optional means it can have
None
value. Required optional means you will get error if you don't provide it, but you can set it toNone
. Not required not optional means there is a default value which is notNone
and you can't set it toNone
also cli changed a bit, now
mlem build *
are subcommands (instead of argument). Alsomlem build
is a separate command (for building from saved config using-l
). I will update this soon when I decide how to better reflect this (same withserve
andapply-remote
)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.
style:
-p <path>
,--project <path>
- path to MLEM ...default - check how it's done in the docs
TEXT should be renamed into path, also in the CLI itself
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.
done, but sometimes the duplication of metavar feels strange