-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
WIP: Add yaml support for stub mapping configuration files #2436
base: master
Are you sure you want to change the base?
WIP: Add yaml support for stub mapping configuration files #2436
Conversation
@karanjamutahi thanks! Could you please summarize how far it was reviewed during the hackfest? Maybe @Mahoney could take a look as a mntor there. Also, please register for hacktoberfest as mentioned in the PR template |
A few questions came up mostly around the many features yaml to test for. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In longer run I would switch to a framework like picocli that makes it easier tmake interexchangeable configs and to embed CLI options there, but it is a good progress towards better flexibility of the code. Still needs to be taken over the line. @karanjamutahi lemme know if any help is needed
Test yaml multi-document format
0f06b14
to
03daf9e
Compare
@karanjamutahi Hi! It would be great to get it over the line. Let me know if I could help somehow |
References
#2411
Submitter checklist
#help-contributing
or a project-specific channel like#wiremock-java