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

Way Of Shadow: exclude caster from darkness effects #5187

Conversation

nelsam
Copy link

@nelsam nelsam commented Jan 31, 2025

Unfortunately, I have no Windows PC (everything I own runs linux), so as far as I know I can't compile this and test it. Based on nothing more than gut instinct, I tried to track down how to exclude the Way of Shadow monk from being blinded by their own darkness. I have only the faintest glimmer of hope that it will work.


Following the 2024 Warrior of Shadow (which seems to be what the current Way of Shadow subclass is built around), the monk should be able to see through their own Darkness ability.

You can see within the spell’s area when you cast it with this feature.

This adds '.ExcludeCaster()' to the Way of Shadow's Darkness ability, to match the 2024 rules.

Following the 2024 Warrior of Shadow (which seems to be what the current
Way of Shadow subclass is built around), the monk should be able to see
through their own Darkness ability.

> You can see within the spell’s area when you cast it with this feature.

This adds '.ExcludeCaster()' to the Way of Shadow's Darkness ability, to
match the 2024 rules.
@ThyWoof
Copy link
Collaborator

ThyWoof commented Feb 2, 2025

unfortunately not that simple.

@ThyWoof ThyWoof closed this Feb 2, 2025
@nelsam
Copy link
Author

nelsam commented Feb 2, 2025

Cool, thanks for taking the time to look anyway. I made the change before I realized that there's (afaict) no way to build on linux, and I thought I might as well submit it. I hope I didn't waste too much of your time.

@ThyWoof
Copy link
Collaborator

ThyWoof commented Feb 2, 2025

It’s forcing me to think out of the box again. Good challenge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants