Skip to content

[pull] master from braydie:master #3

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 27 commits into from
Jan 21, 2025
Merged
Changes from 1 commit
Commits
Show all changes
27 commits
Select commit Hold shift + click to select a range
8be1a12
Create 09-Design Patterns.md
braydie Jan 15, 2016
eb76879
Update 03-How to Manage Consultants.md
luigimartin Apr 26, 2016
4f09b95
Update 03-How to Manage Consultants.md
luigimartin Apr 26, 2016
206995e
Update 03-How to Manage Consultants.md
luigimartin Apr 26, 2016
86a634d
Russian translation. Version 01 (#1)
paveltovchigrechko Jan 3, 2023
2b1cc7a
Russian version. Version 02 (#3)
paveltovchigrechko Jan 5, 2023
b8b947e
Russian version, after review. Version 03 (#4)
paveltovchigrechko Jan 6, 2023
041b3c6
Spell checking, edit non-russian files. Version 04 (#5)
paveltovchigrechko Jan 7, 2023
703bcd3
feat: Support for the Spanish language has been added.
mjaviermurua Dec 4, 2023
3970695
feat: first draft
Jan 7, 2024
ffe7058
Create How-To-Be-Mentored
melanielaporte Jan 20, 2025
2255095
Create How-to-Mentor-Others
melanielaporte Jan 20, 2025
3591e34
Update README.md
melanielaporte Jan 20, 2025
91c0033
Update README.md
melanielaporte Jan 20, 2025
8e6c017
Update How-To-Be-Mentored
melanielaporte Jan 20, 2025
6a85ee6
Update How-To-Be-Mentored
melanielaporte Jan 20, 2025
20945f7
Update How-to-Mentor-Others
melanielaporte Jan 20, 2025
3cb610e
Update How-To-Be-Mentored
melanielaporte Jan 20, 2025
1d719ca
Merge pull request #145 from melanielaporte/master
RobertLRead Jan 21, 2025
a02b120
Merge pull request #142 from burkaslarry/others/apply_traditional_chi…
RobertLRead Jan 21, 2025
760dac5
Merge pull request #140 from maximilianomurua/feat/spanish-translatio…
RobertLRead Jan 21, 2025
30e2fbb
Merge branch 'master' into master
RobertLRead Jan 21, 2025
ccd84ad
Merge pull request #136 from paveltovchigrechko/master
RobertLRead Jan 21, 2025
73d39ac
Merge pull request #78 from braydie/section-design-patterns
RobertLRead Jan 21, 2025
faa27d4
some tweaks in grammar
RobertLRead Jan 21, 2025
8e0f9f8
resolving conflicts
RobertLRead Jan 21, 2025
bc928de
Merge pull request #146 from braydie/BRANCH_NAME
RobertLRead Jan 21, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update 03-How to Manage Consultants.md
  • Loading branch information
luigimartin committed Apr 26, 2016
commit 4f09b95d28c57e16614ae0c4755fd9572f6cbbe5
Original file line number Diff line number Diff line change
Expand Up @@ -11,12 +11,7 @@ Formally you are the “customer”, so you have the right to pretend what you p
Remember that an annoyed consultant will perform very poorly, resulting in an economic damage for your company and a misstep for you as a leader.
If consultants are going to write code, you must review it carefully as you go along. You cannot get to the end of the project with the risk of a large block of code that has not been reviewed. This is true of all team members, really, but you will usually have more knowledge of the team members closer to you.
You should always take into account that writing readable and maintainable code is not the main priority of a consultant - its goal is to meet client expectation which are mostly expressed in terms of deadlines and functionalities that shall be implemented.
When deadlines are approaching, consultant are masters in rapidly patching software to make it work quickly – even if that means to embed into source code something like configuration parameters or fake data. Take a look at this example:

// FIXME
sendMail(request.getAccount().getAccountInternet().getUsr(),
"Please confirm registration by clicking on the following link:<br/><br/><a href=\"http://www.ourdomain.com/abcard/confirmRegistration.html?userId="
+ account.getUserId() + "\">Confirm</a>");
When deadlines are approaching, consultant are masters in rapidly patching software to make it work quickly – even if that means to embed into source code something like configuration parameters or fake data.

This is why you shall explicitly ask consultant to write good code, more than reviewing every single LOC – let pass the message that the extra effort spent for writing elegant and reusable code will be appreciated.
Another good practice is to make every team member accountable in the same way for the work they produce – e.g. make mandatory for everyone to provide header comment blocks into programming code or editor/reviewers tables into delivered documents so that any file can be attributed to its author. Given the current emphasis about online professional reputation and personal branding, accountability will be a lever for consultants to avoid delivering something that they wouldn’t be proud of, encouraged by the fact that they could be already gone somewhere else when poor quality of their job is revealed.
Expand Down