server
server copied to clipboard
LandSandBoat
Hostile herbivores
Basis and evidence are based off the following: https://www.youtube.com/watch?v=2q93do8wQKQ&ab_channel=WiggoCaptures https://drive.google.com/open?id=1lAiUP1bGNvEBVpIFZFshmzjonNK89Cvv
I affirm:
- [x] I have paid attention to this example and will edit again if need be to not break the formatting, or I will be ignored
- [x] I have read and understood the Contributing Guide
- [x] I've tested my code and the things my code has changed since the last commit in the PR, and will test after any later commits
What does this pull request do?
- Added a unique sheep charge ability for the fighting sheep that has no prepare time as seen in the video. mob skill ID based off of live capture.
- Sheep given an auto attack of sheep charge that deals knockback
Steps to test these changes
Enter Hostile Herbivores bcnm in Horlais Peak to test.
would you like me to rebase this for you after I get out of work today?
sure! I've been stumbling through github lmao
Critical has been helping me out a ton. But rebasing and managing dozens of different branches at the same time is difficult
might be worth your while to waste hdd space on full separate clones in diff folders just to keep yourself from accidently mixing the content the way you've been
yeah perhaps! I've been trying to push to ASB as well, so I rebase to push for them. But then I forgot there was still a PR open here, so I should have cloned it. And now it's all a mess lol.
Maybe I'm lacking context here, but you know if you merge something here ASB will get it when they merge upstream (LSB). Is there something different you need to do to cater to ASB? If we solve that problem you won't need to maintain two branches
Well I'm trying to get content pushed into the database asap so I can start throwing testers at things for horizon (I'm implementing all the BCNM / KSNMs) So the sooner I can get things in the better. ASB usually merges my stuff within a few hours and can go straight to the testers. Genuinely curious, and also have very little idea how github works, but can you guys merge my BCNM changes from ASB to LSB using something like cherry-pick? That way the content would be tested and confirmed to be working accurately and it wouldn't require as much work dealing with my terrible code lol
using something like cherry-pick?
that results in conflicts and duplicated commits. its a bad workflow.
So the sooner I can get things in the better.
it's usually the opposite in my experience. the faster things get rammed through the more disasters a server will have. Best of luck to them. >.>
its a really weird situation we're in, we take consideration of you folks on downstream servers that you never have any idea we're covertly saving people from themselves. there are things we could in theory do to bring in code but it would then give the server operator an aneurism next time they pull, so we don't do that. But since they have no idea about it, they often wonder why we can't just xyz. Its unfortunately difficult to communicate all the potential disasters people almost do to themselves daily. I've seen the worst of the worst happen, seen operators throw their hands up and shut down over things they never knew were their own fault. And many a contributor give up after getting their forks in the exact state yours is now.
We try our best to prevent tho.
yeah, no I get it. And I truly don't intend to step on anyone's toes or be a nuisance here. I'm simply over eager to help out and haven't learned common etiquette working with big databases like this.
etiquette working with big ~~databases~~ repo's like this.
This actually isn't that big. messy maybe. We've inherited a lot from years of iteration, both good and bad. Check out unreal engine's repo. That thing is HUGE. Scary huge!
Edit: a word
Should be better now
Vid of what I did: https://www.twitch.tv/videos/1550500996
@dallano You should be good to remove the bit about the TP in the mob script now BTW, the melee skill should not consume tp anymore.
@dallano You should be good to remove the bit about the TP in the mob script now BTW, the melee skill should not consume tp anymore.
nvm, took care of it myself