-
Notifications
You must be signed in to change notification settings - Fork 56
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
remove-hardwired-symbols-and-triggerItem #294
base: master
Are you sure you want to change the base?
remove-hardwired-symbols-and-triggerItem #294
Conversation
9343cc9
to
6fe48fa
Compare
6fe48fa
to
a8613f5
Compare
Cograts on making this compile. Could you try and remove most uses of |
remove-hardwired-symbols-and-triggerItem
a8613f5
to
5bfaee3
Compare
df8be36
to
270c733
Compare
The failed CI seem to be doctests. I guess they just need updating. |
README tells how to run doctests. It may be easier than finding it in the CI script. |
270c733
to
4563085
Compare
Issue LambdaHack#152, removing uses of toContentSymbol where possible
4563085
to
dd995ec
Compare
@@ -263,7 +252,7 @@ dropItems t = ([CmdItemMenu, CmdItem], t, dropCmd) | |||
|
|||
descIs :: [TriggerItem] -> Text | |||
descIs [] = "trigger an item" | |||
descIs (t : _) = makePhrase [tiverb t, tiobject t] | |||
descIs _ = makePhrase ["fling", "in-range projectile"] |
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'd rather inline the whole makePhrase ["fling", "in-range projectile"]
into projectI
, because descIs
is also used in applyIK
so we can't change it freely.
issue 152, remove all hardwired item symbols as well as of TriggerItem. This commit removes the `flingTs` function before continuing to removing the trigger item. `flingTs` added text to `tiverb` and `tiobject`. For now this commit hardwired these texts in the `Game.LambdaHack.Client.UI.Content.Input` function `descIs`
5138a46
to
9aec886
Compare
remove-hardwired-symbols, have not gotten to the the triggeritems yet. All build and test issues resolved.