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

Limit 105mm ROF in CAS strikes #516

Merged
merged 2 commits into from
Jan 19, 2020
Merged

Conversation

LorenLuke
Copy link
Contributor

When merged this pull request will:
Reduce rate of fire when using Blackfin CAS for 105mm gun.

@CreepPork CreepPork added change Issue or PR that will change a implemented feature. priority/low Issue or PR that has no significant impact and does not negatively impact the current user base. labels Jan 3, 2020
@CreepPork
Copy link
Member

At this stage, I am unable to say whether this will be merged into master and eventually make it out to the release build at some point.

The last version release hasn't been made public yet, so there's a high chance your changes will be approved and merged if everything goes well.

@LorenLuke
Copy link
Contributor Author

No worries. Just was something that I always was perturbed about and it seems like it wouldn't hurt much to change it as it's a very specific thing to change. Just hoping it gets included at some point.

Co-Authored-By: Ralfs Garkaklis <ralfs@garkaklis.com>
@CreepPork CreepPork added the testing/not tested Indicates that the PR's latest commit has not been tested and needs testing before merging. label Jan 8, 2020
@Kexanone Kexanone merged commit c26b9ca into ArmaAchilles:master Jan 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
change Issue or PR that will change a implemented feature. priority/low Issue or PR that has no significant impact and does not negatively impact the current user base. testing/not tested Indicates that the PR's latest commit has not been tested and needs testing before merging.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants