Hey folks!
First of all, I want to thank Unity for this awesome initiative and I’m looking forward to contribute as much as I can to this project.
Since I didn’t find any non-assigned task, I wanted to check the project’s hiearchy and see if anything is wrong.
I found that we have a folder called “Tests” which contains an InputTesting Scene and this obviously doesn’t go along with the guidelines : Don’t add testing scenes, testing scripts, and in general assets that you should keep to yourself. Use them to test/prototype, and then exclude them when you push.
I know it’s not a big deal for now, but I just wanted to mention this as early as possible so we don’t get more of these tests on the main project.
I made a PR where i removed the test scene safely without any errors the scene as i think was probably for testing and had only a Input scene object with a script that was not there as well for some reason either way here is the PR.
But what about test scenes for more complex systems, like Inventory or StateMachine?
They’re necessary for other people to see how system is working, to test and develop upon them.