-
Notifications
You must be signed in to change notification settings - Fork 155
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
Blood Cult DLC 1: Make It an Actually Playable Game (Mode) #1276
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: Remuchi <[email protected]>
Signed-off-by: Remuchi <[email protected]>
Signed-off-by: Remuchi <[email protected]>
Signed-off-by: Remuchi <[email protected]>
Signed-off-by: Remuchi <[email protected]>
Signed-off-by: Remuchi <[email protected]>
Signed-off-by: Remuchi <[email protected]>
Q1: Does offering rune only turn people into gems? Can you not convert? N1: Shackles just straight up don't work. I hope this fixes them. |
Bugs reported from the last Deepstation round (some of these might have been fixed already):
|
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
This is starting to grow conflicts. I am going to make a merge with the conflict and if necessary open a new PR to merge this as is. We've been running this on Pirata for a while now and although we haven't been able to test bloodcult as a gamemode, at round start yet, due to other unrelated issues and more recently not enough population, we have noticed no issue which would justify not merging this. About the issue where people were just being transformed in soul shards: After talking with author she explained that ghosted or disconnected people, or people with mindshield are not supposed to be convertible while they are in such status. After this explanation I am yet to see behavior which I think is actually a bug. I wanted to make more contributions to bloodcult but haven't been able to but I can help with getting this merged before it gets stale. Another thing which is confusing about the comments here is it's not clear if they're just comments about bloodcult content in general or about a server running this early merged. So TLDR; I recommend merging this as it does not break the game, has important bugfixes and is prone to getting stale. |
There. I will now test if the merge did not introduce any new weirdness. |
If you prefer I can PR it directly here instead. |
Tests are ok. Converting still works after the merge, which was expected. Throwing cultist items work as they should, I.E: Cultists can throw after new throwing changes, while non-Cultists get stunned when they try. As a side note, converting dionas is not possible which seems intended. I'm not sure what's the species white/blacklist. |
Signed-off-by: Remuchi <[email protected]>
Signed-off-by: Remuchi <[email protected]>
Content.Server/WhiteDream/BloodCult/BloodRites/BloodRitesSystem.cs
Outdated
Show resolved
Hide resolved
Content.Server/WhiteDream/BloodCult/Runes/CultRuneBaseSystem.cs
Outdated
Show resolved
Hide resolved
Co-authored-by: VMSolidus <[email protected]> Signed-off-by: Remuchi <[email protected]>
Heisentest? |
Signed-off-by: Remuchi <[email protected]>
…nstein-Engines into feat/blood-cult-dlc-1
Signed-off-by: Remuchi <[email protected]>
I hate obscure test fails |
I simply need to specify these thing properly in the Yaml but I have no time until tomorrow |
Signed-off-by: Remuchi <[email protected]>
Description
You don't actually have to pay for it, y'know?
TODO
Media
Changelog
🆑