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

✅ test: fix Test_Ctx_ParamParser route param #2119

Merged
merged 1 commit into from
Sep 25, 2022

Conversation

li-jin-gou
Copy link
Contributor

@li-jin-gou li-jin-gou commented Sep 25, 2022

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.
Explain the details for making this change. What existing problem does the pull request solve?

modify test file

Please delete options that are not relevant.

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

Checklist:

  • I have performed a self-review of my own code
  • New and existing unit tests pass locally with my changes

Commit formatting:

Use emojis on commit messages so it provides an easy way of identifying the purpose or intention of a commit. Check out the emoji cheatsheet here: https://gitmoji.carloscuesta.me/

@li-jin-gou li-jin-gou changed the title test: fix Test_Ctx_ParamParser route param 🐛 test: fix Test_Ctx_ParamParser route param Sep 25, 2022
@li-jin-gou li-jin-gou changed the title 🐛 test: fix Test_Ctx_ParamParser route param ✅ test: fix Test_Ctx_ParamParser route param Sep 25, 2022
@efectn efectn merged commit 14a5f61 into gofiber:master Sep 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants