Skip to content

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.

3 participants