Skip to content

Working on comments #49

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

Merged
merged 9 commits into from
Oct 31, 2022
Merged

Working on comments #49

merged 9 commits into from
Oct 31, 2022

Conversation

Rinzii
Copy link
Member

@Rinzii Rinzii commented Oct 28, 2022

Currently working on the commenting across the entire project. Gonna take a bit so I'm gonna keep this as an ongoing PR. @karnkaul if you see comments that are incorrect or if you have suggestions to make them better as I continue to work on the PR lmk. ^^

@Rinzii Rinzii added documentation Improvements or additions to documentation good first issue Good for newcomers Large Issue labels Oct 28, 2022
@Rinzii Rinzii added this to the Hello collab milestone Oct 28, 2022
@karnkaul karnkaul force-pushed the commenting-cleanup branch 2 times, most recently from 2349551 to 4383814 Compare October 29, 2022 01:17
- Remove rect.hpp (unused).
- `Transform` was initially more complex: contained child transforms too. Since it is now a pure data component agnostic to the scene, it has been moved to util.
@karnkaul
Copy link
Member

@Rinzii It would be prudent to merge this in and continue the remaining work in another PR.

@karnkaul
Copy link
Member

Force merging due to inactivity.

@karnkaul karnkaul marked this pull request as ready for review October 31, 2022 02:39
@karnkaul karnkaul merged commit 6171e99 into main Oct 31, 2022
@karnkaul karnkaul deleted the commenting-cleanup branch October 31, 2022 02:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation good first issue Good for newcomers Large Issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants