-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Remove additional/redundant case let as Date. #3123
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
@swift-ci Please test |
That's a mighty log file. Error seems unrelated though.
|
FYI, that test reads from a custom binary format, naught to do with JSON. Timing in tests is notoriously difficult, running again is probably the quickest option. |
@swift-ci Please test Linux platform |
1 similar comment
@swift-ci Please test Linux platform |
Different error on latest, a compile error;
Unrelated again. Pure curiosity, any docs on the testing methodology? |
I'm not sure which you mean, but some docs are available such as below:
|
@swift-ci Please test Linux platform |
@d-ronnqvist @ethan-kusters @franklinsch (Sorry, I'm not sure whom I should request for opinions.) Logs:
|
@YOCKOW Thanks for those links. I'm sure to be biased, I wouldn't be after testing downstream projects like this, for example, so I wanted to read about the testing strategy. Docs look comprehensive, I'll have a read later tonight, thanks again. |
swiftlang/swift-docc#70 should resolve the CI failure. (Thanks to @d-ronnqvist ) |
@swift-ci Please test Linux platform |
Case accidentally included twice, remove a copy.