-
Notifications
You must be signed in to change notification settings - Fork 101
[BUG] Unable to extract version info error #38
Comments
@brandedoutcast please check my pr |
I had the same problem! |
The same problem occurred to me, and i fixed it by explicitly placing the default regex (" (. *) <\ / Version>") in the "VERSION_REGEX" parameter, as was said by @alexvolchetsky |
@alexvolchetsky I'm happy to make Thanks a lot for your contribution 🌟 |
hello @alexvolchetsky, in my case, v2.5.3 still runs into the problem of not recognizing the version number correctly in the context of the full XML of the I tried the default RegEx, When I try the previous regex with v2.5.3, it works OK. See example RegEx here. But the new RegEx does not. It seems to be having problems matching in the context of the full XML body of the Here is an example of the previous regex, |
hello @anibalvelarde , can you please try to use latest v2.5.4 and remove VERSION_REGEX: (.*)</Version> from your pipeline? That might solve your issue |
Hello @alexvolchetsky, I was able to re-try with v2.5.4. Happy to report it worked like a charm! Thanks! |
Description
Recently started receiving errors during running of action.
It says that it can't find version info.
Noticed that in 2.5.3 release VERSION_REGEX were changed to
^<Version>(.*)<\/Version>$
which doesn't ignore whitespaces in the beginning of the lineFailed Action Log URL
https://github.com/alexvolchetsky/yandex.alice.sdk/runs/864428591?check_suite_focus=true
To Reproduce
Test of
^<Version>(.*)<\/Version>$
on content of my .cspoj file:https://regex101.com/r/ZpzZkk/3/
Can't match anything
Expected Behavior
Test of
^\s*<Version>(.*)<\/Version>$
on content of my .cspoj file:https://regex101.com/r/ZpzZkk/2/
Matches version info
The same regex works if set VERSION_REGEX explicitly. For instance:
Environment:
The text was updated successfully, but these errors were encountered: