Remove skip prefix from AI move scores

[jordanmoore753]

Possible naming improvement in the engine/battle/ai/scoring.asm file. I'm making some refactors to this file for my own game where the AI is a bit stronger and noticed these subroutines that use skip as a prefix. I thought the word skip was misleading since in all of the routines they are not skipping that action they are prefixing; in fact, they are all attempting to do the action that their name suggests they are skipping.

I know that the name is intended to show that they are skipping the discouragement or encouragement from the regular flow of the routine that calls it, but I still found the word skip misleading.

I considered also switching the skip for maybe but I don't think that works as it looks like maybe is used as a convention when some chance is involved with AI_50_50 or one of those similar routines is used. I felt just using discourage or encourage kept the naming more consistent within the file instead of using skip or maybe for these examples.

If people disagree with my take on this, no worries! 👍