Skip to content

Conversation

@dscho
Copy link
Member

@dscho dscho commented Apr 14, 2020

I noticed that the GitHub workflow failed. This is my attempt at fixing it.

dscho added 2 commits April 14, 2020 22:45
The test expected a verbatim match, but the output is localized.
The test case wants to compare potentially localized output, so it needs
to use `test_i18ncmp`.
Copy link

@jeffhostetler jeffhostetler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dscho
Copy link
Member Author

dscho commented Apr 14, 2020

FTR I turned off the branch protection, as we no longer have an Azure Pipelines definition, just aGitHub workflow. Unfortunately, it seems that this PR still thinks that the Pipeline needs to run.

@dscho
Copy link
Member Author

dscho commented Apr 14, 2020

Unfortunately, it seems that this PR still thinks that the Pipeline needs to run.

I guess this comment did it: now things are green again.

@dscho dscho merged commit 390cb49 into microsoft:vfs-2.26.1 Apr 15, 2020
@dscho dscho deleted the fix-v2.26.1-test-failures branch April 15, 2020 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants