Skip to content
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

WFLY-3419 Only one jar in a module is scanned for beans.xml #6860

Merged
merged 2 commits into from
Oct 31, 2014

Conversation

stuartwdouglas
Copy link
Contributor

No description provided.

@stuartwdouglas
Copy link
Contributor Author

@jharting does this look ok to you?

@wildfly-ci
Copy link

Linux with security manager Build 308 is now running using a merge of 743347f

@wildfly-ci
Copy link

Linux Build 5316 is now running using a merge of 743347f

@wildfly-ci
Copy link

Windows Build 424 is now running using a merge of 743347f

@wildfly-ci
Copy link

Linux with security manager Build 308 outcome was FAILURE using a merge of 743347f
Summary: Tests failed: 2, passed: 792, ignored: 244 Build time: 0:06:26

Build problems:

Failed tests detected

Failed tests

org.jboss.as.test.smoke.jms.SendToTopicFromWithinTransactionTest.sendSuccessfully: java.lang.AssertionError: null
    at org.junit.Assert.fail(Assert.java:86)
    at org.junit.Assert.assertTrue(Assert.java:41)

org.jboss.as.test.smoke.jms.SendToQueueFromWithinTransactionTest.sendSuccessfully: java.lang.AssertionError: null
    at org.junit.Assert.fail(Assert.java:86)
    at org.junit.Assert.assertTrue(Assert.java:41)

@wildfly-ci
Copy link

Windows Build 424 outcome was SUCCESS using a merge of 743347f
Summary: Tests passed: 3025, ignored: 238 Build time: 0:55:02

@wildfly-ci
Copy link

Linux Build 5316 outcome was SUCCESS using a merge of 743347f
Summary: Tests passed: 3024, ignored: 238, muted: 1 Build time: 0:57:12

@bstansberry bstansberry added ready-for-merge Only for use by those with merge permissions! and removed ready-for-merge Only for use by those with merge permissions! labels Oct 27, 2014
@bstansberry
Copy link
Contributor

Oops. just saw the question for @jharting

@jharting
Copy link
Contributor

@stuartwdouglas The change looks good to me. Apologies for the delay.

@bstansberry bstansberry added the ready-for-merge Only for use by those with merge permissions! label Oct 31, 2014
bstansberry added a commit that referenced this pull request Oct 31, 2014
WFLY-3419 Only one jar in a module is scanned for beans.xml
@bstansberry bstansberry merged commit b0d6429 into wildfly:master Oct 31, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-merge Only for use by those with merge permissions!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants