Fix the link for "Build (for testing PR)" in appVeyor message #17510
+1
−1
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Link to issue number:
Probably fix-up of #17214.
Follow-up of #17249.
Summary of the issue:
When an appVeyor test failure message is issued, the build link in this message is on
l10nUtil.exe
rather than onnvda_snapshot_xxxx.exe
.Description of user facing changes
The NVDA snapshot executable will be again downloadable from appVeyor messages.
Description of development approach
Refine the filtering to find the snapshot executable on "nvda_.exe" rather than on ".exe", as done in the deploy script in #17249 in appVeyor script.
Testing strategy:
Introduce an error in unit tests to trigger the message and check the result: see #17510 (comment).
Known issues with pull request:
None
Code Review Checklist:
@coderabbitai summary