Skip to content

/Cataclysm-DDA

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

Rail-mount crossbows didn't actually spawn. This fixes that. #6109

Merged
merged 1 commit into from Feb 15, 2014

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
5 participants
@Rivet-the-Zombie @NaturesWitness @kevingranade @KA101 @Soyweiser
@Rivet-the-Zombie
Copy link
Member

commented Feb 14, 2014

No description provided.

@NaturesWitness

This comment has been minimized.

Sign in to view
Copy link
Contributor

commented Feb 14, 2014

Did we ever get all the bugs fixed in these?
@Rivet-the-Zombie

This comment has been minimized.

Sign in to view
Copy link
Member Author

commented Feb 14, 2014

I wasn't aware they were bugged.
@kevingranade

This comment has been minimized.

Sign in to view
Copy link
Member

commented Feb 14, 2014

There have intermittently been issues with mounted ranged weapons all
along. People update the main ranged code but miss the mounted weapon code.
@Rivet-the-Zombie

This comment has been minimized.

Sign in to view
Copy link
Member Author

commented Feb 14, 2014

Ah. Well in that case I believe they're okay. For now.

At least as far as the last time I checked, all the aux-weapons were working.
@KA101

This comment has been minimized.

Sign in to view
Copy link
Contributor

commented Feb 15, 2014

Yeah, I'm pretty confident BevapDin fixed that one a while ago.

KA101 added a commit that referenced this pull request Feb 15, 2014

@KA101
Merge pull request #6109 from Rivet-the-Zombie/rail-crossbow-spawn 
Rail-mount crossbows didn't actually spawn. This fixes that.
e8eede7

@KA101 KA101 merged commit e8eede7 into CleverRaven:master Feb 15, 2014
1 check passed

1 check passed

default
Details

@Rivet-the-Zombie Rivet-the-Zombie deleted the Rivet-the-Zombie:rail-crossbow-spawn branch Feb 16, 2014

origamiwolf added a commit to origamiwolf/DinoCataclysm that referenced this pull request Feb 16, 2014

@KA101 @origamiwolf
Merge pull request CleverRaven#6109 from Rivet-the-Zombie/rail-crossb… 
…ow-spawn

Rail-mount crossbows didn't actually spawn. This fixes that.
fc4a3cb
@Soyweiser

This comment has been minimized.

Sign in to view
Copy link
Contributor

commented Feb 17, 2014

http://smf.cataclysmdda.com/index.php?topic=5388.0 This topic suggests they are perhaps still broken.
@KA101

This comment has been minimized.

Sign in to view
Copy link
Contributor

commented Feb 17, 2014

I think that was loading the main gun. Feel free to open a new issue, rather than posting in a merged PR?

(Unless you're asking to revert this, and I'm thinking you're not?)
@Soyweiser

This comment has been minimized.

Sign in to view
Copy link
Contributor

commented Feb 17, 2014

Well, I have not looked into the whole issue with rail weapons. And it sounded related. Not sure if it was. This seemed like a logical place for people who looked into the rail weapons issue to see it. But if it is a different bug, I'll open a different issue.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.