-
-
Notifications
You must be signed in to change notification settings - Fork 469
Allow client peds to enter/exit client vehicles #3678
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
Conversation
vendor/google-breakpad/src/client/linux/handler/minidump_descriptor.h
Outdated
Show resolved
Hide resolved
Shouldn't we also use the |
yep, it works here too, it's tested |
All done |
@tederis review please ✨ |
Why do you ask for a review if you don't acknowledge it? You're not following the coding guidelines and you're marking conversations as resolved even though you haven't done anything 🤷 |
I acknowledge you may have reading problems but I made everything you asked for in conversations, just didn't answer to it and pushed commit |
Also I just want to add you're able to make comments and reviews but you're not one deciding here if pull request should be merged or not, thanks 🙏 |
Actually, you didnt. You pointlessly resolved conversations without adding the requested changes |
I did in commit e461e70, I'm sorry for your blindness, other changes are approved by fileex. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR already looks good, which unfortunately cannot be said about the others
Being rude to others does not benefit you whatsoever. You only added the type but did not remove the braces; i.e. You acknowledged 1 out of 3 conversations. |
@gownosatana Generally speaking, over all your recent PR's and code reviews on it, you are creating a lot of unnecesary drama, and you don't want to agree to do things the way this project has implemented. You are this close to an organisation (repository) ban and the deletion of all your PR's, so im just warning you to keep yourself in check. Do not reply to this comment. Start showing some more respect, and cut the drama/combativeness. You're not gonna get things the way you want it; the world or this project doesn't revolve around you. If you don't like that, you are free to just go away. |
This comment was marked as abuse.
This comment was marked as abuse.
This PR is being closed for the same reason as the comment explains on the other one closed. |
This pull request allows client created peds to enter client created vehicles.
setPedEnterVehicle(Element ped, [Element vehicle, bool passenger])
When no vehicle is specified ped will enter closest client vehicle
Test resource: test.zip