-
Notifications
You must be signed in to change notification settings - Fork 6
feat: extended cli #251
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
base: main
Are you sure you want to change the base?
feat: extended cli #251
Conversation
2cc0e11 to
34af17d
Compare
docs/source/worker/index.md
Outdated
| ## Worker concepts | ||
|
|
||
| Workers provide non-trivial functionality to a Tierkreis graph. | ||
| Writing a worker |
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.
Is this meant to be an incomplete sentence or a link to other docs?
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.
Or maybe a title?
| └── uv.lock | ||
| ``` | ||
|
|
||
| The repository is structure is intended to separate _graphs_, _workers_ and library code. |
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 guess we use the word graphs in a lot of places but do we want to call this folder workflows or similar instead? I guess it would be inconsistent with what we've done in other places.
| │ │ └── main.py | ||
| │ └── workers/ | ||
| │ └── example_worker/ | ||
| │ ├── api/ |
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 presumably is where we get going to generate stubs for the worker?
Make a collection of tasks available through cli:
This is only the first set of changes