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

Dog Whistle Affects More Dogs #23794

Merged
merged 1 commit into from May 21, 2018

Conversation

Projects
None yet
3 participants
@DracoGriffin
Copy link
Member

commented May 21, 2018

Dog whistle will now affect creatures with DOGFOOD flag rather than a single creature.

Simple PR to allow for upcoming expanded dog breeds and more.

Change target of dog whistles
Dog whistle will now affect targets with DOGFOOD flag
@ZhilkinSerg

This comment has been minimized.

Copy link
Contributor

commented May 21, 2018

Jenkins rebuild.

@ZhilkinSerg ZhilkinSerg merged commit e8166a0 into CleverRaven:master May 21, 2018

0 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
gorgon-ghprb Build triggered. sha1 is merged.
Details

@DracoGriffin DracoGriffin deleted the DracoGriffin:dogwhistle_moretargets branch May 21, 2018

@AMurkin

This comment has been minimized.

Copy link
Contributor

commented May 21, 2018

Also check hearing senses of a critter?

@DracoGriffin

This comment has been minimized.

Copy link
Member Author

commented May 22, 2018

Also check hearing senses of a critter?

I only adjusted to match the dog food code for upcoming PR. This is probably something that should be developed further (hearing check) but I'm not sure if critters (as in tamed dogs in this example) can be deafened. If so, then it'd be worth looking into (I have basically no knowledge regarding dog whistles and hearing with dogs -- and how it affects them if deaf/hearing impaired).

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.