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.

Sign up for GitHub

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

Jump to bottom

compatibleMagazines - Add optional bool to get all muzzles #1123

Merged
merged 3 commits into from
Apr 24, 2019
Merged

compatibleMagazines - Add optional bool to get all muzzles #1123

merged 3 commits into from
Apr 24, 2019

Conversation

PabstMirror
Copy link
Contributor

No description provided.

commy2
commy2 approved these changes Apr 22, 2019
View reviewed changes
Copy link
Contributor

@commy2 commy2 left a comment

Choose a reason for hiding this comment

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

Apart from preInit caching quirks, works fine.

commy2 and others added 2 commits April 23, 2019 12:46
@commy2 @PabstMirror
Apply suggestions from code review
134406d 
Co-Authored-By: PabstMirror <pabstmirror@gmail.com>
@PabstMirror
fix github code review messing up
7ecbb5b
@CBATeam CBATeam deleted a comment from PabstMirror Apr 23, 2019
@commy2 commy2 merged commit 45564d0 into master Apr 24, 2019
@commy2 commy2 deleted the compatibleMagsAllMuzzles branch April 24, 2019 14:18
@commy2
Copy link
Contributor

commy2 commented Apr 24, 2019

ref #1122

@commy2 commy2 added this to the 3.11 milestone Apr 24, 2019
@commy2 commy2 added the Bug Fix label Apr 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@commy2 commy2 commy2 approved these changes

@dedmen dedmen dedmen approved these changes

Assignees
No one assigned
Labels
Bug Fix
Projects
None yet
Milestone
3.11
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@PabstMirror @commy2 @dedmen