loteque / wasteland-warrior

3 stars 0 forks source link

add pipe_pistol to PlayerController #161

Closed loteque closed 11 months ago

loteque commented 11 months ago

https://github.com/loteque/wasteland-warrior/assets/69282314/f0d45969-d036-4afc-97ba-9fb3380d735e

https://github.com/loteque/wasteland-warrior/assets/69282314/dba8116e-ff72-444a-839c-783c36dfa679

jonathandavidlewis commented 11 months ago

@loteque Oh, can you change the title of the PR so its more meaningful when viewed in the PR list later?

loteque commented 11 months ago

Name: PipePistol.tsn instead of pipe_pistol.tscn

I dig the guns directory. Other than that, works for me!

This was indeed very easy to review. Bravo! 💯

I was thinking we should use snake_case for tscn files that are not meant to be reusable or inheritable. What do you think?