90 karma · joined December 26, 2008
That being said, this plugin still adds some additional features: - Shows names of PRs instead of just numbers - Shows issues (and not just PRs) that reference the commit
Towards the end of me being there, they added a more senior developer/manager to the Romanian team -- his code was much better (i.e. not dreamweaver generated HTML/CSS) and was easier to communicate with, but still the code quality from the others on the team was very unmaintainable.
I apologize if it seemed as if I was generalizing the entire country of Romania's programming abilities -- definitely not what I intended. If all the developers were as good as the senior developer added on later, I'm sure it would have went much better.
My main point remains the same though, and I think is one that we can both agree on - that hiring inexpensive labor with only focusing on the front end user experience, without any respect or idea of how well the back end is being constructed is a bad idea. I'm sure outsourcing can work, but only with careful attention to the behind the scenes backend code quality, to ensure the project is not being held together by duct tape.
The HTML and CSS they wrote were generated using Dreamweaver, and their PHP was a mess with tons of copying and pasting. It was impossible to get the company's higher level management team to understand why continuing to use these guys was bad idea, because they saw the front end 'just working', yet it was so hard for us to modify and understand any of their code.
If you intended to have a site that one day has skilled programmers working on it and is actually maintainable, be careful about things 'just working' with outsourced code.