Skip to content

Conversation

@quickfur
Copy link
Member

@quickfur quickfur commented Jun 7, 2017

No description provided.

@JackStouffer JackStouffer added Merge:auto-merge Review:Trivial typos, formatting, comments labels Jun 7, 2017
@JackStouffer
Copy link
Contributor

@quickfur looks like a transient error on the doc builder, please force push

@quickfur
Copy link
Member Author

quickfur commented Jun 7, 2017

Force push isn't doing anything, because my branch is already up-to-date. Any other way to kick the autotester to re-run the test?

@wilzbach
Copy link
Contributor

wilzbach commented Jun 7, 2017

Force push isn't doing anything, because my branch is already up-to-date

Do something like this

git commit --amend --no-edit

(On the go, otherwise I can repush myself later).

@quickfur
Copy link
Member Author

quickfur commented Jun 7, 2017

Ah, thanks for the tip! All done now.

@quickfur
Copy link
Member Author

quickfur commented Jun 7, 2017

Hrm. It's still failing. :-(

@JackStouffer
Copy link
Contributor

Ping @CyberShadow, build log says

ddmd/arrayop.d(21): Error: module id is in file 'ddmd/id.d' which cannot be read

@CyberShadow
Copy link
Member

Yeah, DAutoTest is currently all red. It greenlit a bad PR which should've been red due to a cached dependency. Sorry about that. Currently bisecting to find the bad PR.

@andralex
Copy link
Member

andralex commented Jun 7, 2017

@CyberShadow thx! Also @CyberShadow and @MartinNowak why is jenkins unhappy with this? Error msg seems unrelated:

source/unit_threaded/io.d(334,26): Error: expected 0 arguments, not 2 for non-variadic function type nothrow @nogc ref @system File()
source/unit_threaded/io.d(335,26): Error: expected 0 arguments, not 2 for non-variadic function type nothrow @nogc ref @system File()
source/unit_threaded/io.d(285,27): Error: template instance unit_threaded.io.threadWriter!(makeGlobal, makeGlobal) error instantiating
Warning: testIssue31 passes the criteria for a value-parameterized test function but doesn't have the appropriate value UDAs.
         Consider changing its name or annotating it with @DontTest

@CyberShadow
Copy link
Member

why is jenkins unhappy with this

I think Jenkins has been unhappy about everything since #5421.

@CyberShadow
Copy link
Member

Fixed, should be green soon: dlang/dlang.org#1682

@andralex andralex merged commit 8bb9c3c into dlang:master Jun 8, 2017
@quickfur quickfur deleted the std_traits_style branch June 8, 2017 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Review:Trivial typos, formatting, comments

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants