search

ome / omero-ms-queue

GNU General Public License v3.0
3 stars 5 forks source link

Smuggler v1.1.0-beta changeset #2

Closed c0c0n3 closed 7 years ago

c0c0n3 commented 7 years ago

This pull request brings in the changes from smuggler v1.0.0 to v1.1.0-beta.

There were no changes w/r/t user functionality but the build system and all third-party libs have been upgraded to the latest & greatest :-) In particular, we're now using Artemis instead of HornetQ. Details over here.

codecov-io commented 7 years ago

Codecov Report

Merging #2 into master will increase coverage by 0.42%. The diff coverage is 86.36%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master       #2      +/-   ##
============================================
+ Coverage     81.75%   82.18%   +0.42%     
- Complexity     1256     1385     +129     
============================================
  Files           287      286       -1     
  Lines          3804     3929     +125     
  Branches        177      184       +7     
============================================
+ Hits           3110     3229     +119     
- Misses          589      592       +3     
- Partials        105      108       +3
Impacted Files Coverage Δ Complexity Δ
...java/ome/smuggler/providers/q/ServerConnector.java 100% <ø> (ø) 3 <0> (ø) :arrow_down:
...me/smuggler/config/wiring/omero/SessionQBeans.java 100% <ø> (ø) 6 <0> (ø) :arrow_down:
.../ome/smuggler/providers/q/CountedScheduleSink.java 100% <ø> (ø) 3 <0> (+1) :arrow_up:
...c/main/java/ome/smuggler/providers/q/Messages.java 94.11% <ø> (ø) 9 <0> (+1) :arrow_up:
...smuggler/config/wiring/imports/ImportGcQBeans.java 100% <ø> (ø) 4 <0> (ø) :arrow_down:
...e/smuggler/config/wiring/imports/ImportQBeans.java 100% <ø> (ø) 4 <0> (ø) :arrow_down:
...va/ome/smuggler/config/wiring/mail/MailQBeans.java 100% <ø> (ø) 6 <0> (ø) :arrow_down:
.../java/ome/smuggler/providers/q/QueueConnector.java 75% <ø> (ø) 4 <0> (ø) :arrow_down:
...va/ome/smuggler/providers/q/MessageBodyWriter.java 100% <ø> (ø) 2 <0> (ø) :arrow_down:
...va/ome/smuggler/providers/q/MessageBodyReader.java 100% <ø> (ø) 2 <0> (ø) :arrow_down:
... and 48 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 18e555a...ea4d3bf. Read the comment docs.

jburel commented 7 years ago

Discussed today with @c0c0n3 Merging and tagging at 0.2.0 so @c0c0n3 can move forward with the refactoring @c0c0n3 will create issues to detail the refactoring steps.