Conversation
|
Automatic reply from Jenkins: Can I test this? |
|
Jenkins: test this please |
|
Refer to this link for build results (access rights to CI server needed): |
|
would be great, but I'd love to see a dedicated unit test for this before we merged this in are you willing to give this a shot, @geimer? let me know if you need any help I should've asked you the same for #1164, so if you're up for it, add one for |
|
Ah, wait, #1164 isn't merged yet, so the unit tests for |
|
Jenkins: ok to test |
|
@geimer: looks great, thanks for tackling the unit test too! I'll merge this in if Jenkins is happy with it. |
|
Refer to this link for build results (access rights to CI server needed): |
|
Going in, thanks @geimer! |
The
path_matchesroutine should ignore non-existing paths rather than fail hard...