Skip to content
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

added 1up token #946

Merged
merged 2 commits into from
Mar 11, 2013
Merged

added 1up token #946

merged 2 commits into from
Mar 11, 2013

Conversation

NimbusBP1729
Copy link
Member

@jhoff asked me to add these.

They are only found in-game when you kill bats, which I usually do with punching.

We might still need a sound effect.

with #939 it'll mean you can place 1ups anywhere you please

@jhoff
Copy link
Member

jhoff commented Mar 9, 2013

I think the bat probability is wrong. Don't you want it to drop health, occasionally dropping an extra life? It's currently the other way around.

@NimbusBP1729
Copy link
Member Author

killing bats is hard for people. originally i was gonna have it only drop 1ups. it was to be a treat, since we have no other ways of getting 1ups.

@NimbusBP1729
Copy link
Member Author

i am now on your side. I'll switch the probabilities accordingly. I don't want the game to be too easy.

EDIT: meh... I'm thinking it over. You take a lot of risk when provoking a bat. I say this after having returned to the level and getting hit instead of hitting.

@kyleconroy
Copy link
Member

Let's make the bat drop 1 ups all the time, since they are hard to kill. We'll eventually change this, but for now I think it's a good idea.

@NimbusBP1729
Copy link
Member Author

additionally it's relatively easy to kill bats with 0-gravity projectiles like the icicles.

@jhoff
Copy link
Member

jhoff commented Mar 11, 2013

#939 has been merged

kyleconroy added a commit that referenced this pull request Mar 11, 2013
@kyleconroy kyleconroy merged commit b729294 into hawkthorne:master Mar 11, 2013
@NimbusBP1729 NimbusBP1729 deleted the life_token branch March 11, 2013 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants