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

Update README.md #508

Merged
merged 3 commits into from
Jan 29, 2025
Merged

Update README.md #508

merged 3 commits into from
Jan 29, 2025

Conversation

elijahbenizzy
Copy link
Contributor

@elijahbenizzy elijahbenizzy commented Jan 29, 2025

Fixes inconsistencies in the REAMDE example found when showing Burr to some folks


Important

Fix inconsistencies in README.md example code for human_input() and ai_response() functions.

  • Example Code:
    • In human_input(), added example code for creating chat_item.
    • In ai_response(), added comment about querying LLM and corrected chat_item creation.

This description was created by Ellipsis for a1052dc. It will automatically update as commits are pushed.

Fixes inconsistencies in the REAMDE example
@elijahbenizzy elijahbenizzy marked this pull request as ready for review January 29, 2025 04:43
Copy link

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

❌ Changes requested. Reviewed everything up to a1052dc in 28 seconds

More details
  • Looked at 21 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 0 drafted comments based on config settings.

Workflow ID: wflow_26yt3CFDAFKifaIM


Want Ellipsis to fix these issues? Tag @ellipsis-dev in a comment. You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

README.md Show resolved Hide resolved
Copy link

github-actions bot commented Jan 29, 2025

A preview of a1052dc is uploaded and can be seen here:

https://burr.dagworks.io/pull/508

Changes may take a few minutes to propagate. Since this is a preview of production, content with draft: true will not be rendered. The source is here: https://github.com/DAGWorks-Inc/burr/tree/gh-pages/pull/508/

@elijahbenizzy elijahbenizzy merged commit acbeff5 into main Jan 29, 2025
10 of 11 checks passed
@elijahbenizzy elijahbenizzy deleted the update-readme-example branch January 29, 2025 04:58
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