Skip to content

fix: api doc example error#3917

Closed
StudyingLover wants to merge 1 commit into
langgenius:deploy/devfrom
StudyingLover:deploy/dev
Closed

fix: api doc example error#3917
StudyingLover wants to merge 1 commit into
langgenius:deploy/devfrom
StudyingLover:deploy/dev

Conversation

@StudyingLover

@StudyingLover StudyingLover commented Apr 27, 2024

Copy link
Copy Markdown
Contributor

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Fixes # (issue)

Type of Change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • TODO

Suggested Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings
  • I ran dev/reformat(backend) and cd web && npx lint-staged(frontend) to appease the lint gods

@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Apr 27, 2024
@StudyingLover
StudyingLover changed the base branch from deploy/dev to main April 27, 2024 07:36
@dosubot dosubot Bot added size:XL This PR changes 500-999 lines, ignoring generated files. and removed size:XS This PR changes 0-9 lines, ignoring generated files. labels Apr 27, 2024
@StudyingLover
StudyingLover changed the base branch from main to deploy/dev April 27, 2024 07:36
@dosubot dosubot Bot added size:XS This PR changes 0-9 lines, ignoring generated files. and removed size:XL This PR changes 500-999 lines, ignoring generated files. labels Apr 27, 2024

@takatost takatost left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Apr 27, 2024
@StudyingLover

Copy link
Copy Markdown
Contributor Author

emmmm, I am a fresh man for this project, what should I do , because I find pytets failed

@takatost takatost left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Oops...the branch you submitted is deploy/dev, please branch out from the main branch before submitting. 👀

@dosubot dosubot Bot removed the lgtm This PR has been approved by a maintainer label Apr 27, 2024
@takatost

Copy link
Copy Markdown
Contributor

I've closed it for you, please resubmit.

@takatost takatost closed this Apr 27, 2024
@StudyingLover

Copy link
Copy Markdown
Contributor Author

ooo! thanks, I resubmit #3925

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants