Skip to content

Conversation

tylerslaton
Copy link
Contributor

@tylerslaton tylerslaton commented Feb 23, 2024

First pass at some documentation around what tools are, how they work, and how you can make them. This also include some restructuring to make content flow more naturally.

To test this out, run make serve-docs.

At the time of creating this PR, the packaging format for external tools (as we want it to look) has not been implemented. This PR reflects the current state of things.

Copy link
Member

@njhale njhale left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

Could you also update the copyright year in the README.md?

https://github.com/gptscript-ai/gptscript?tab=readme-ov-file#license

it still reads 2023

@tylerslaton tylerslaton force-pushed the tool-docs branch 2 times, most recently from 274b742 to af0d2a3 Compare February 26, 2024 21:54

```yaml
description: This is a tool that calls a shell script.
args: arg1: The first argument.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we use the first argument? Its not clear if it is being used or not to the reader.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe that this is addressed everywhere now!

@tylerslaton tylerslaton force-pushed the tool-docs branch 5 times, most recently from 669c121 to 247f28e Compare February 27, 2024 17:16
- chore: add note about adding local model support
- docs: add image-generator cookbook
- docs: add gen-docs.gpt and a make target for it

Signed-off-by: tylerslaton <[email protected]>
@tylerslaton tylerslaton merged commit e4229ce into gptscript-ai:main Feb 27, 2024
@tylerslaton tylerslaton deleted the tool-docs branch February 27, 2024 17:31
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.

4 participants