The rules tradeskill skillup modifiers should probably be Real instead of Int

[wmrojer]

Seeing that the variable the rules for tradeskill skillup modifiers is read into is a float and the comment "// Remember: skillup_modifier is (float). Lower is better" in the code and the way the skillup chance is calculated the rules should probably be stored and read as RULE_REAL instead of RULE_INT. Default values are between 1 and 4 and the note says lower is better. So not many different steps to increase the chance without the rule being a float.

The affected rules are: TradeskillUpAlchemy, TradeskillUpBaking, TradeskillUpBlacksmithing, TradeskillUpBrewing, TradeskillUpFletching, TradeskillUpJewelcrafting, TradeskillUpMakePoison, TradeskillUpPottery, TradeskillUpResearch, TradeskillUpTinkering.

[wmrojer]

And there isn't even a rule for Tailoring. Maybe add that one or it's hard coded to 2.0

[Aeadoin]

Resolved in #2674, Tailoring looks to have been added as well.