-
-
Notifications
You must be signed in to change notification settings - Fork 36
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
Add support for geometryFunction
prop to RDraw
#261
base: main
Are you sure you want to change the base?
Conversation
@mmomtchev could you have a quick second to look at this? |
You don't have any unit tests, try to cover at least the basic use case Also, I wonder if there is any need for this, but anyway For PRs the rules are that you are cited:
Otherwise, if you meet code coverage standards, your PR will get mered |
Sounds good! I'll try to get some tests written in my free time. |
Hi @mmomtchev again, I've had some down time over the last months and only now had some free time to add a basic unit test. Let me know if there's anything else you'd like to me change before getting this merged. |
As you may have seen from my profile, I am currently unemployed and living on social welfare because of an extortion on a galactic scale, involving the French and EU justice system, the French police, the French bar association, and a number of large IT companies such as Google, Facebook, Twitter and Microsoft. In order to intimidate me, people are posting simultaneously on my projects. Yours is one of these comments. |
@mmomtchev It would be really great if this could be merged in -- it would be super cool to be able to use your package and work 🚀 |
@WillUK99 I received a simultaneous call with your second message from the French administration stating that I will be evicted from my apartment next week. Obviously, nothing in this situation bothers you. You seem to be one of these persons who are deterred from crime only by severe punishment, who does not have any limits, morals or dignity. You and the whole geospatial community have chosen to take part in a criminal extortion. I personally refuse to live in a world where this kind of extortion, involving state institution, is possible and I will never give up my fight. Even if I understand that nothing will happen to you until I have discourage the state from taking part in a crime, I still think that your choice will eventually have consequences. |
@WillUK99 I know that everything I said will fall on deaf ears with someone like you, but since I understand where you stand, you should also know that the reason for this is not because someone is bothered to be a criminal in public, the reason is because he was bothered by this situation: |
@mmomtchev what are you on about? Could we keep this professional and not accuse people of being apart of your personal challenges? This PR is quite small and it makes me really uneasy to have to annoy someone to approve it. Let me know if there's anything you need in order to get this merged. |
Expose
geometryFunction
prop to enable usingcreateBox()
function or any custom algorithms in user-land.