Skip to content
This repository has been archived by the owner on May 26, 2020. It is now read-only.

Break the dependency on the event-store #16

Merged
merged 1 commit into from
Feb 20, 2019
Merged

Conversation

wlzhn
Copy link
Collaborator

@wlzhn wlzhn commented Feb 20, 2019

No description provided.

@coveralls
Copy link

coveralls commented Feb 20, 2019

Pull Request Test Coverage Report for Build 133

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 90.018%

Totals Coverage Status
Change from base Build 126: 0.0%
Covered Lines: 487
Relevant Lines: 541

💛 - Coveralls

<version>${event-store.version}</version>
<groupId>org.apache.tomee</groupId>
<artifactId>apache-tomee</artifactId>
<version>${apache-tomee.version}</version>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The version should be brought by the common-bom

<artifactId>test-utils-persistence</artifactId>
<version>${event-store.version}</version>
<groupId>org.apache.tomee</groupId>
<artifactId>apache-tomee</artifactId>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you need all apache-tomee artefacts? Usually they are used in our IT's and we mostly use openejb-core. Please take a look at the framework.

@wlzhn wlzhn changed the title TP-1082 Break the dependency on the event-store Break the dependency on the event-store Feb 20, 2019
@MartinYSpasov MartinYSpasov merged commit a14bc46 into master Feb 20, 2019
@MartinYSpasov MartinYSpasov deleted the dev/tp-1082 branch February 20, 2019 15:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants