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

Fix tests and adds support for custom schema #164

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

at-silva
Copy link

@at-silva at-silva commented Nov 3, 2023

feat(api): adds support for custom schema

  • Adds support for custom schema i.e.:
    type (
        Config struct {
      	  Path string `schema:"{'required': true, 'starts_with': '/'}"`
        }
    )

fix(test): typo and coroutine deadlock

@CLAassistant
Copy link

CLAassistant commented Nov 3, 2023

CLA assistant check
All committers have signed the CLA.

@at-silva at-silva changed the title fix(test): typo and coroutine deadlock Fix tests and adds support for custom schema Nov 3, 2023
@gszr
Copy link
Member

gszr commented Mar 13, 2024

I like the annotations usage to describe richer schemas. Thanks for opening this. We will look into this feature as soon as we can. I am downgrading this one to "draft" until we can approach this -- also taking requests like #93 into consideration.

@gszr gszr marked this pull request as draft March 13, 2024 19:10
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.

TestEnv.ClientRes.Status is not behaving as expected when ExitStatus is called.
3 participants