-
-
Notifications
You must be signed in to change notification settings - Fork 411
std::string_view posix tests #2347
Conversation
|
Thanks for your pull request and interest in making D better, @TurkeyMan! We are looking forward to reviewing it, and you should be hearing from a maintainer soon.
Please see CONTRIBUTING.md for more information. If you have addressed all reviews or aren't sure how to proceed, don't hesitate to ping us with a simple comment. Bugzilla referencesYour PR doesn't reference any Bugzilla issue. If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog. Testing this PR locallyIf you don't have a local development environment setup, you can use Digger to test this PR: dub fetch digger
dub run digger -- build "master + druntime#2347" |
|
I don't run these systems... if people can help me get these across the line, that'd be real great! |
You need to start 😉. |
|
I remember something similar to that output not too long ago, can I suggest you rebase? If nothing else it will rule that out. |
I would see Apple burn before I infect any piece of hardware in my presence! ;) |
The PR title says "posix". There are other Posix systems than those created by Apple 😉. Beside the smiley I added above, it would gain you a lot to test on non Windows platforms locally. To avoid having to wait for the CI systems to pass or for someone else to fix your PR. |
True, but Apple has caused the the most trouble, with having a slightly weird ecosystem; gotta explicitly request to link the C++ runtime (even when supplying
I mean, I could just stop doing this altogether... I'm not getting anything at all for my time from this at this point! My stuff already works fine. |
|
Tests are broken due to |
You could just not run the test for older compilers (or on the auto-tester). Since a few days, buildkite runs the full druntime test suite too (see e.g. dlang/ci#301) |
Alternatively you could always ping @braddr to upgrade the compilers on these machines or ping him here https://github.com/braddr/d-tester/issues. |
4020ad9 to
79f4cf4
Compare
No description provided.