-
Notifications
You must be signed in to change notification settings - Fork 142
Allow URLs with capital letters #833
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
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>
The Java validator allows URLs with capitals. |
Thanks for catching and fixing this! Would you mind adding appropriate test cases to the |
Humm, I would need some help with the testing @armintaenzertng . It looks like if I run pytest in my dev branch it is not running my modified code. I did pip3 install the local folder before, but still it seems that my changes are not running. Can you help me to figure this out? |
@meretp can you help me with this testing issue? |
@CsatariGergely sure, what exactly did you try? I don't see a test on this branch so I can't really reproduce your issue right now, did you follow the instructions from the Contributing.md file? |
also, it would be good if you could rebase your changes on main and not merge the changes from main into this branch to keep the history clean |
Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>
Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>
@meretp Thanks I think I've figured out the tests. I just did not know where to add an URI with capitals to test. |
@meretp , @armintaenzertng what else is missing to get this merged? |
Fixes: #832