-
Notifications
You must be signed in to change notification settings - Fork 83
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
feat: neg operator #388
feat: neg operator #388
Conversation
Merge Develop into Main
Merge Develop in Main
Merge Develop into Main
Merge main into Develop
@raphaelDkhn I couldn't find any contribution guidelines in the repo. Please let me know if any docs/linting/tests, etc need to be done before PR is merged. Thanks |
Thanks @0x73e, I will review later today. Here is the contribution guideline. Here you can find a step-by-step tuto to contribute to Orion. If I see anything missing during my review, I let you know :) |
Thanks, totally missed this |
LGTM! Good first PR @0x73e thanks! Just three small things that was missing:
|
@all-contributors add @0x73e for code |
I've put up a pull request to add @0x73e! 🎉 |
Pull Request type
Closes #362
Please check the type of change your PR introduces:
What is the current behavior?
Missing Neg operator.
Issue Number: N/A
What is the new behavior?
Implements Neg operator.
Does this introduce a breaking change?
Other information