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

feat(go): Handling of Errors in go with GenkitError #2643

Draft
wants to merge 11 commits into
base: main
Choose a base branch
from
Draft

feat(go): Handling of Errors in go with GenkitError #2643

wants to merge 11 commits into from

Conversation

sahdev77
Copy link
Contributor

@sahdev77 sahdev77 commented Apr 4, 2025 •
edited
Loading

Addition of new Genkit error in go:

  • Handling Error using Genkit Error.
  • Removal of earlier defined Genkit Error in jsonschema.
  • Removal of HTTPError from misc.
    Resolving : 2476
    Checklist (if applicable):
  • PR title is following https://www.conventionalcommits.org/en/v1.0.0/
  • Tested (manually, unit tested, etc.)
  • Docs updated (updated docs or a docs bug required)

@github-actions github-actions bot added the go label Apr 4, 2025
@pavelgj pavelgj requested a review from apascal07 April 7, 2025 12:45
@sahdev77 sahdev77 changed the title Improve errors throughout feat(go): Handling of Errors in go with GenkitError Apr 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@apascal07 apascal07 Awaiting requested review from apascal07

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
go tooling
Projects
Genkit Backlog
Status: No status
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@sahdev77