-
Notifications
You must be signed in to change notification settings - Fork 53
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
feat: support to set the prefix of mock server #402
Conversation
|
Coverage summary from CodacySee diff coverage on Codacy
Coverage variation details
Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: Diff coverage details
Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: See your quality gate settings Change summary preferencesCodacy will stop sending the deprecated coverage status from June 5th, 2024. Learn more |
There are 20 test cases, failed count 0: See all test records
Reported by api-testing. |
There are 23 test cases, failed count 0: See all test records
e2e-testing |
hi @YukiCoco, please feel free to help to review this PR. Thanks. |
I just saw the update that the PR has been successfully merged and closed—thanks for taking care of it! I must apologize for missing your initial message in my crowded email inbox. I'm truly sorry for not being able to provide my input on time. I’d be grateful for any opportunity to contribute to future projects. Please don't hesitate to reach out. |
No worry. Please let me know if you find any things that need to be improved in this PR. By the way, there are many things we can do on the mock server part. For example:
|
I have tested this code and it works well. I will try to support more mock functions. |
Co-authored-by: rick <linuxsuren@users.noreply.github.com>
Co-authored-by: rick <linuxsuren@users.noreply.github.com>
Co-authored-by: rick <linuxsuren@users.noreply.github.com>
What type of PR is this?
What this PR does / why we need it:
Allowing user to set the prefix can meet more use cases.
Which issue(s) this PR fixes:
Fixes #