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

add api_test to generate more test cases #452

Merged
merged 2 commits into from
Mar 2, 2025

Conversation

Luckfort
Copy link
Contributor

@Luckfort Luckfort commented Mar 2, 2025

Thank you for your contribution to AIOS!
Before submitting the pull request, please make sure to read and follow requirements below and then you can replace all the texts with your own messages. This helps improve the efficiency of the review process.

Remember to update documentation

When adding new features or updating existing functions, please review the documentation to determine if any changes are required. If so, ensure that the documentation is updated accordingly. The source code for the documentation is located at: https://github.com/agiresearch/AIOS/tree/main/docs. Instructions for writing documentation can be found here: https://github.com/agiresearch/AIOS/blob/main/docs/README.md.

Write informative PR title and classification

Only specific types of PRs will be reviewed. The PR title is prefixed appropriately (i.e., "prefix: description") to indicate the type of change. Please use one of the prefixs as below:

  • feat Add new features
  • fix Fix bugs
  • docs Modify documents like README, CONTRIBUTE
  • style Modify code format like space and comma without changing code logic
  • refactor Refactor code structure without adding new features or fixing new bugs
  • perf Improve performance or user experience
  • test Test features, including unit test and integration test
  • chore Change the build procedure or add dependencies
  • revert Revert to the previous version

Wait for the reviews

Copy link
Collaborator

@dongyuanjushi dongyuanjushi left a comment

Choose a reason for hiding this comment

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

@dongyuanjushi dongyuanjushi merged commit 87d60ed into agiresearch:main Mar 2, 2025
1 check failed
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.

2 participants