This is an archive of the discontinued LLVM Phabricator instance.

[clang-tidy] Fix windows tests
ClosedPublic

Authored by njames93 on Jan 2 2021, 3:53 PM.

Details

Summary

Attempt to fix the 2 failing tests identifier in 48646.
Appears that python3 doesn't like nested double quotes in single quoted strings, hopefully nested single quotes in double quoted strings is a-ok.

Diff Detail

Event Timeline

njames93 created this revision.Jan 2 2021, 3:53 PM
njames93 requested review of this revision.Jan 2 2021, 3:53 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 2 2021, 3:53 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript

I don't have a (reliable) windows machine to test so can you take a look please

thakis accepted this revision.Jan 3 2021, 3:04 PM

Hey, thanks!

I don't have a win machine at hand either, but let's land this and see what the win bot on http://45.33.8.238/ thinks about this. It should take at most 15 min to cycle.

This revision is now accepted and ready to land.Jan 3 2021, 3:04 PM
This revision was automatically updated to reflect the committed changes.

http://45.33.8.238/win/30678/summary.html It looks like it works, as nothing needed to be built it was a v fast build

thakis added a comment.Jan 3 2021, 5:07 PM

Looks like this worked :)

Thanks again!