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

Adds new mob: Space Laser Beam Carp #451

Merged
merged 4 commits into from
Apr 16, 2014
Merged

Conversation

Dorsisdwarf
Copy link
Contributor

Space Laser Beam Carp are introduced as a child mob of spacecarp
-Attacks with laser beams instead of biting
-Modified interaction messages
-Custom-made sprite, edited from current spacecarp images.
-Currently only possible to encounter via admin intervention
layzorcerp
EDIT: Image was taken before the carp were made to actually fire lasers.

@Dorsisdwarf Dorsisdwarf reopened this Apr 16, 2014
rapid = 1
retreat_distance = 5
minimum_distance = 5
weapon1 = /obj/item/weapon/gun/energy/laser/captain
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this weapon1 var is irrelevant.
All the weapon var does is make the mob drop this weapon in it's Die() proc, which you've already overriden to just delete the mob.

You can remove the var/weapon1 and the assigning of the captains laser to it
The mob can shoot WITHOUT the need of this weapon var.

@RemieRichards
Copy link
Contributor

Once you've done as I've asked then the Code is good to go.

@Dorsisdwarf
Copy link
Contributor Author

Removed the var and the weapon. Thanks for clarifying that it wasn't needed.

@GunHog
Copy link
Contributor

GunHog commented Apr 16, 2014

You are a monster for making Laser carp. I love it. I hope you make it more than just admin only though. Maybe a prob(5) chance whenever normal carp spawn!

@RemieRichards
Copy link
Contributor

Right, Code is solid, I'll try to get someone with Maintainer access to sort this now.

@dannno dannno merged commit aa16e50 into NTStation:master Apr 16, 2014
@Iamgoofball
Copy link
Contributor

I LOVE YOU

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.

4 participants