Congrats on your first addon, lets take a look at the code for a moment:
function servercmdrevive(%client, %tar) {
%tar = findclientbyname(%tar);
//we want to check if the player is admin so he can't go apestuff and revive his whole team
// Try to steer away from ugly looking code, just use four-spaced tabs, and make a new line before & after braces
if(!%client.isAdmin) {
return; }
//is the target even existent? are we even existent? are we a dream?
if(!isObject(%tar)){
messageclient(%client,'',"<color:ff0000>Target is invalid");
return;
}
// An easier, and better method is to just use %tar.player.instantRespawn()
%tar.setdead(0);
%tar.setlives(-1);
%tar.spawnplayer();
// Its best to try to use \cN, where N can be anything from 0-9 for some pre-defined colors
// For example: \c0 is red and \c3 is green. You should only use hexadecimal color values for very specify colors.
// You can type /colortest in-game to see all the color codes.
messageall('',"<color:ffffff>" @ %tar.name SPC "<color:00ff00>was revived by<color:ffffff>" SPC %client.name);
//anti-abuse measure
echo(%client.name SPC "revived" SPC %tar.name);
}
My comments are bolded. Just keep practicing, and don't make bad habits. Good luck.