-
Notifications
You must be signed in to change notification settings - Fork 241
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
EquipTest Updates #568
EquipTest Updates #568
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One regression I would prefer fixed. Otherwise, looks good.
@@ -18,8 +18,8 @@ namespace ACE.Command.Handlers | |||
{ | |||
internal enum TestWeenieClassIds : uint | |||
{ | |||
Pants = 120, | |||
Tunic = 134, | |||
Pants = 120, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't understand why you shifted these to non indented. We've been keeping the assignments aligned on enums, I dislike the inconsistency this adds.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't change this purposefully, I think Visual Studio must've made this change for me when I did something. Weird. Fixed it, anyways.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
VS has been changing this automatically for me as well, if you use the CTRL+K and then CTRL+D shortcut, it will show you the same results...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whoever merged the changelog is an idiot, otherwise great stuff! (totally not writing this b/c github needs an approval...)
Changed EquipTest debug command to utilize the index of a clothing table item and added an optional shade parameter