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

feat: port machine gun #954

Merged
merged 9 commits into from
Apr 4, 2024
Merged

Conversation

DRuppFv
Copy link
Contributor

@DRuppFv DRuppFv commented Apr 3, 2024

This PR closes #573.

I added bullet_spread system to the machine gun because I think it balances the game and looks good, and added empty_cooldown because it would be really boring if gun_empty.ogg kept playing at the same frequency as the normal cooldown, so in this gun we have the shooting cooldown and the empty sound cooldown. I also made a very slight change in the main system for using items because it had to be able to shoot constantly, so now you can keep shoot input pressed with any weapon.

Something I think is important to mention is that the shooting animation is synchronized to the rate of fire, so if you change the rate of fire the animation will also change. It could be easily changed with a new variable in the .yaml so let me know whether you think it should be faster or slower.

The machine gun has two states (idle and shooting) and its only function is to reset and start animations.

Cargo.lock Outdated Show resolved Hide resolved
@zicklag zicklag mentioned this pull request Apr 4, 2024
@MaxCWhitehead
Copy link
Collaborator

@DRuppFv It looks like the continuous integration found some issues with the formatting in machine_gun.rs, could you run cargo fmt and update this?

Thank you!

Copy link
Collaborator

@MaxCWhitehead MaxCWhitehead left a comment

Choose a reason for hiding this comment

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

Thank you very much for your work here @DRuppFv :)

@MaxCWhitehead MaxCWhitehead added this pull request to the merge queue Apr 4, 2024
Merged via the queue into fishfolk:main with commit 8fb37e1 Apr 4, 2024
8 checks passed
@DRuppFv DRuppFv deleted the machinegun_port branch April 4, 2024 04:48
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.

Re-implement Machine Gun
2 participants