Skip to content
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

ivec += and -= functions #373

Merged
merged 4 commits into from
Dec 25, 2023
Merged

ivec += and -= functions #373

merged 4 commits into from
Dec 25, 2023

Conversation

BeeverFeever
Copy link
Contributor

@recp This first commit is purely just to make sure the format and style is what you were thinking of for the ivec functions. I noticed the += family of functions weren't even there so I implemented them as well. This is all I have done, I haven't bothered with tests yet, I am just seeking your feedback on any details before I implement them for ivec3 and ivec4 (and also finish ivec2). There may be errors in here but I will read over it all when I polish it up.

@recp
Copy link
Owner

recp commented Dec 11, 2023

@BeeverFeever many thanks, looks good, waiting for other types 👍

@recp
Copy link
Owner

recp commented Dec 22, 2023

@BeeverFeever ping.

@BeeverFeever
Copy link
Contributor Author

@recp Hey, sorry for the wait. I havent been home for a while so I havent been able to do anything. Im getting home tomorrow today though. I should be able to have it almost done by tomorrow.

@recp
Copy link
Owner

recp commented Dec 22, 2023

@BeeverFeever thanks 🤗

@BeeverFeever
Copy link
Contributor Author

@recp Just finished them (atleast I think everything is done).

@recp recp merged commit 40cfbe9 into recp:master Dec 25, 2023
49 checks passed
@recp
Copy link
Owner

recp commented Dec 25, 2023

@BeeverFeever many thanks for your contributions, the PR is merged 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants