Public bug reported:
There are lots of small spelling mistakes throughout the code : datas=data,
childs=children, logistic=logistics etc.
I think these deserve to be fixed one day.
The point of correct English in the code is:
- well-written code is a documentation in itself
- you can't search for misspelled words in IDEs
- bad English is a clue to find code that was never reviewed nor refactored
- respect for your fellow developers worldwide
It's not risky because:
- renaming variables and functions is easily and quickly done (ecplise even has
a wizard for it)
- bzr lets you review what you merge
- yml tests will catch the problems
Lionel
Related bugs : Bug #326160, Bug #327259
** Affects: openobject-addons
Importance: Undecided
Status: New
** Summary changed:
- "Children" not "childs" in the code
+ Spelling errors in the code
--
Spelling errors in the code
https://bugs.launchpad.net/bugs/673087
You received this bug notification because you are a member of C2C
OERPScenario, which is subscribed to the OpenERP Project Group.
Status in OpenObject Addons Modules: New
Bug description:
There are lots of small spelling mistakes throughout the code : datas=data,
childs=children, logistic=logistics etc.
I think these deserve to be fixed one day.
The point of correct English in the code is:
- well-written code is a documentation in itself
- you can't search for misspelled words in IDEs
- bad English is a clue to find code that was never reviewed nor refactored
- respect for your fellow developers worldwide
It's not risky because:
- renaming variables and functions is easily and quickly done (ecplise even has
a wizard for it)
- bzr lets you review what you merge
- yml tests will catch the problems
Lionel
Related bugs : Bug #326160, Bug #327259
_______________________________________________
Mailing list: https://launchpad.net/~c2c-oerpscenario
Post to : [email protected]
Unsubscribe : https://launchpad.net/~c2c-oerpscenario
More help : https://help.launchpad.net/ListHelp
