Change in variables' names

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

Change in variables' names

achryso
Since last night, someone changed the variable userName of Broker into
username.

This may have been the reason why the tests fail since morning.

It should be preferrable to inform the rest of the Development team about
changes like this, because problems may be created.

Thank you in advance for your understanding.

Antonios

Reply | Threaded
Open this post in threaded view
|

Re: Change in variables' names

ddauer
Well the change would have happened either way, but I'll let you know next time.

If you haven't read the commit notes, the change was necessary to be able to integrate user-management into the web-app.

David

On Fri, Mar 18, 2011 at 11:33 AM, achryso [via Power TAC Developers] <[hidden email]> wrote:
Since last night, someone changed the variable userName of Broker into
username.

This may have been the reason why the tests fail since morning.

It should be preferrable to inform the rest of the Development team about
changes like this, because problems may be created.

Thank you in advance for your understanding.

Antonios




If you reply to this email, your message will be added to the discussion below:
http://power-tac-developers.975333.n3.nabble.com/Change-in-variables-names-tp2697427p2697427.html
To start a new topic under Power TAC Developers, email [hidden email]
To unsubscribe from Power TAC Developers, click here.