[ 
http://jira.codehaus.org/browse/CONTINUUM-2597?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=247648#action_247648
 ] 

Wendy Smoak commented on CONTINUUM-2597:
----------------------------------------

The patch has a lot of noise (reformatting/whitespace changes) that makes it 
difficult to see what you're actually changing.

(Do we have standard code style settings available for contributors?)

Can you please add some documentation explaining what this method returns?

I think I also saw a TODO in there and a method returning null... did something 
get left out?

> Webservice for getting platform of build agent
> ----------------------------------------------
>
>                 Key: CONTINUUM-2597
>                 URL: http://jira.codehaus.org/browse/CONTINUUM-2597
>             Project: Continuum
>          Issue Type: New Feature
>            Reporter: Mark John Kenedy L. Magallanes
>         Attachments: CONTINUUM-2597.patch
>
>
> Right now we don't have a web service expose for build agent in retrieving 
> it's platform. It would be useful to have this feature to prepare for future 
> implementation that would involve sorting build agents by platforms.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to