billylam / wdio-wdiov5testrail-reporter

https://www.npmjs.com/package/wdio-wdiov5testrail-reporter
MIT License
6 stars 8 forks source link

Log invalid case ids when strictCaseMatching set to true #20

Closed hdesai-dave closed 1 year ago

hdesai-dave commented 1 year ago

@billylam any thoughts here? Can we get the above PR merged?

billylam commented 1 year ago

Thanks for the reminder. I'll take a look. Unfortunately my npm 2fa is failing for some reason so publishing might be a few days from now.

hdesai-dave commented 1 year ago

Thanks for the reminder. I'll take a look. Unfortunately my npm 2fa is failing for some reason so publishing might be a few days from now.

If we can get the pr reviewed meanwhile, that would be great!

billylam commented 1 year ago

I have already reviewed and left comments.

hdesai-dave commented 1 year ago

I have already reviewed and left comments.

Do not see any comments https://github.com/billylam/wdio-wdiov5testrail-reporter/pull/21/files https://github.com/shubham0803 do you see on your end?

shubham0803 commented 1 year ago

I have already reviewed and left comments.

Do not see any comments https://github.com/billylam/wdio-wdiov5testrail-reporter/pull/21/files https://github.com/shubham0803 do you see on your end?

I am also not able to see any review comments on the PR

billylam commented 1 year ago

Are you looking at the conversation tab? https://github.com/billylam/wdio-wdiov5testrail-reporter/pull/21

image
hdesai-dave commented 1 year ago

Are you looking at the conversation tab? #21

image

Your comment Status is pending. Need to submit.

billylam commented 1 year ago

Ah, thanks.