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.

Sign up for GitHub

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

Jump to bottom

fix: update doc tutorials #11

Merged
merged 1 commit into from
Dec 9, 2024
Merged

fix: update doc tutorials #11

merged 1 commit into from
Dec 9, 2024

Conversation

Joshua-Briggs
Copy link
Member

the main changes are

  • when making a router you used to initialize them with source parameter, that has since changed to sources which takes a list rather then a singular node.
  • all the node functions need to use the async def rather then async.
  • when calling the graph you need to use await

@Joshua-Briggs Joshua-Briggs self-assigned this Dec 9, 2024
jamescalam
jamescalam approved these changes Dec 9, 2024
View reviewed changes
Copy link
Member

@jamescalam jamescalam left a comment

Choose a reason for hiding this comment

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

lgtm thanks @Joshua-Briggs

@jamescalam jamescalam changed the title Updated the tutorials in the docs so they should all work without errors now fix: update doc tutorials Dec 9, 2024
@jamescalam jamescalam merged commit b14c824 into main Dec 9, 2024
@jamescalam jamescalam deleted the josh/updated-tutorials branch December 9, 2024 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jamescalam jamescalam jamescalam approved these changes

Assignees

@Joshua-Briggs Joshua-Briggs

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Joshua-Briggs @jamescalam