-
Notifications
You must be signed in to change notification settings - Fork 85
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
Fix the patch #11
Fix the patch #11
Conversation
Hi,
|
I haven't tested the module yet, but I will Also I feel like these are not errors but just warnings wrongly labelled |
i havent used your patch and created a few days earlier a new patch and i have the same problem like @dd667 that the ahbot is not selling anything |
Yeah nothing works, or i don't know how to test it but i can still merge this at least |
it doesnt work but you merge? :D is that the way to go? |
feel free to revert to before this PR then, it will solve all your problems and don't talk to me if it doesn't |
the patch itself works. but the rest of the mod doesnt. |
Seems like you made a mistake in your patch too TobiMiller, like me Fixed on latest commit on master now. |
I didn't compile with this yet, but
git apply
worked fineCloses #8