Moti Asayag has posted comments on this change.

Change subject: core: Add hamcrest-library as an explicit test dependency
......................................................................


Patch Set 1: Code-Review+2

(1 comment)

https://gerrit.ovirt.org/#/c/40362/1/backend/manager/modules/bll/pom.xml
File backend/manager/modules/bll/pom.xml:

Line 185:          <groupId>org.reflections</groupId>
Line 186:          <artifactId>reflections</artifactId>
Line 187:          <version>${reflections.version}</version>
Line 188:      </dependency>
Line 189:       <dependency>
we should add it also to DAL project where it can also be used.

doesn't required to be part of this patch.
Line 190:           <groupId>org.hamcrest</groupId>
Line 191:           <artifactId>hamcrest-library</artifactId>
Line 192:           <version>${hamcrest.version}</version>
Line 193:           <scope>test</scope>


-- 
To view, visit https://gerrit.ovirt.org/40362
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idbec3ea20820972e1daf4981846290c40163b257
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yevgeny Zaspitsky <[email protected]>
Gerrit-Reviewer: Martin PeÅ™ina <[email protected]>
Gerrit-Reviewer: Moti Asayag <[email protected]>
Gerrit-Reviewer: Yevgeny Zaspitsky <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to