Menu
▾
▴
taverna-code — Code review
You can subscribe to this list here.
| 2010 |
Jan
|
Feb
|
Mar
|
Apr
|
May
|
Jun
(5) |
Jul
|
Aug
|
Sep
|
Oct
|
Nov
|
Dec
|
|---|---|---|---|---|---|---|---|---|---|---|---|---|
| 2011 |
Jan
|
Feb
|
Mar
|
Apr
|
May
(1) |
Jun
|
Jul
(5) |
Aug
|
Sep
|
Oct
|
Nov
|
Dec
|
| 2012 |
Jan
|
Feb
|
Mar
|
Apr
|
May
(2) |
Jun
|
Jul
|
Aug
|
Sep
|
Oct
|
Nov
|
Dec
|
|
From: <ta...@go...> - 2012-05-09 09:47:30
|
Comment by st...@my...: Score: Negative General Comment: See http://taverna.googlecode.com/svn/taverna/taverna-parent/tags/taverna-parent-2.3.0/pom.xml for correct list For more information: http://code.google.com/p/taverna/source/detail?r=14777 |
|
From: <ta...@go...> - 2012-05-09 09:44:35
|
Comment by st...@my...: Score: Negative General Comment: This "Fix" broke the repositories. Plugin developers now get: [ERROR] Failed to execute goal on project astrotaverna-workbench-test: Could not resolve dependencies for project uk.org.taverna.astro:astrotaverna-workbench-test:jar:0.4-SNAPSHOT: Failure to find org.openprovenance:opm-core:jar:1.1.8 in http://www.mygrid.org.uk/maven/repository was cached in the local repository, resolution will not be reattempted until the update interval of mygrid-repository has elapsed or updates are forced -> [Help 1] For more information: http://code.google.com/p/taverna/source/detail?r=14777 |
|
From: <ta...@go...> - 2011-07-12 15:10:56
|
Comment by so...@gm...: Score: Negative General Comment: This was a mistake, sorry. For more information: http://code.google.com/p/taverna/source/detail?r=13557 |
|
From: <ta...@go...> - 2011-07-12 15:06:41
|
Comment by so...@gm...: Score: Negative General Comment: T2-1949 For more information: http://code.google.com/p/taverna/source/detail?r=13557 |
|
From: <ta...@go...> - 2011-07-12 15:02:39
|
Comment by so...@gm...: Score: Negative General Comment: This was a mistake, sorry.. For more information: http://code.google.com/p/taverna/source/detail?r=13556 |
|
From: <ta...@go...> - 2011-07-06 11:14:21
|
Comment by st...@so...: Score: Neutral General Comment: Not sure if this should make it into the 2.3.. does not sound like a blocker to me. For more information: http://code.google.com/p/taverna/source/detail?r=13506 |
|
From: <ta...@go...> - 2011-07-05 13:38:32
|
Comment by st...@my...: Score: Negative General Comment: Should check all external references - not just pick the first one (which might also be UNKNOWN) For more information: http://code.google.com/p/taverna/source/detail?r=13496 |
|
From: <ta...@go...> - 2011-05-06 09:53:44
|
Comment by st...@my...: Score: Negative General Comment: This skips every second item as the finally-block also removes the item. This removes the possibility to stop the thread. This removes the item from the queue before it's deleted, so that persisting the deletion queue on shutdown will not persist the head item. For more information: http://code.google.com/p/taverna/source/detail?r=13038 |
|
From: <ta...@go...> - 2010-06-21 10:32:19
|
st...@my... commented on revision r10972 in project taverna. Details are at http://code.google.com/p/taverna/source/detail?r=10972 Score: Positive General Comment: Fair enough :) Respond to these comments at http://code.google.com/p/taverna/source/detail?r=10972 -- You received this message because you starred this review, or because your project has directed all notifications to a mailing list that you subscribe to. You may adjust your review notification preferences at: http://code.google.com/hosting/settings |
|
From: <ta...@go...> - 2010-06-21 10:28:16
|
don...@ma... commented on revision r10973 in project taverna. Details are at http://code.google.com/p/taverna/source/detail?r=10973 Score: Positive General Comment: Put it down to experimentation. ;-) Respond to these comments at http://code.google.com/p/taverna/source/detail?r=10973 -- You received this message because you starred this review, or because your project has directed all notifications to a mailing list that you subscribe to. You may adjust your review notification preferences at: http://code.google.com/hosting/settings |
|
From: <ta...@go...> - 2010-06-21 10:24:07
|
don...@ma... commented on revision r10972 in project taverna. Details are at http://code.google.com/p/taverna/source/detail?r=10972 General Comment: IllegalStateException is unchecked and general. IllegalStateTransitionException is specific and checked. Respond to these comments at http://code.google.com/p/taverna/source/detail?r=10972 -- You received this message because you starred this review, or because your project has directed all notifications to a mailing list that you subscribe to. You may adjust your review notification preferences at: http://code.google.com/hosting/settings |
|
From: <ta...@go...> - 2010-06-21 09:36:22
|
st...@my... commented on revision r10972 in project taverna. Details are at http://code.google.com/p/taverna/source/detail?r=10972 Score: Neutral General Comment: Why a new IllegalStateTransitionException when there is already java.lang.IllegalStateException ..? Respond to these comments at http://code.google.com/p/taverna/source/detail?r=10972 -- You received this message because you starred this review, or because your project has directed all notifications to a mailing list that you subscribe to. You may adjust your review notification preferences at: http://code.google.com/hosting/settings |
|
From: <ta...@go...> - 2010-06-21 09:32:17
|
st...@my... commented on revision r10973 in project taverna. Details are at http://code.google.com/p/taverna/source/detail?r=10973 Score: Positive General Comment: You can't really vote for your own commits! :-) Respond to these comments at http://code.google.com/p/taverna/source/detail?r=10973 -- You received this message because you starred this review, or because your project has directed all notifications to a mailing list that you subscribe to. You may adjust your review notification preferences at: http://code.google.com/hosting/settings |
