-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Set the environmental tools/guide for the project with dotfiles, possible a small bash install script #346
Comments
Thanks @studentutu, there are some good suggestions. Some we already have in the project, like locking the Unity version or asset naming conventions. Some others we'll do of course, like compressing assets... Some things I don't agree with. For instance, using .jpg in the project is a very bad idea. Same for .oggs. You want clean files in the project, then they will be compressed in the build. But if you include compressed files as sources, sure, the project size goes down, but what if you have to work on them to tweak them? It's kind of bad practice. Same for the Changelog generator... you mean for eventual final release of the game? I feel like the changelog for a game (especially of this size) needs to be curated rather than automated. Although we could implement it anyway to have a reference, and then throw away some of the text 🤷🏻♂️ |
Completely agree regarding compressed files as sources. |
Can you link an example of one? To understand the level of detail that you're thinking about, and whether it is useful. |
@ciro-unity here is a good example (it is automated, created based on the specific tags in the commit message) https://github.com/ExtendRealityLtd/Zinnia.Unity/blob/master/CHANGELOG.md |
Really cool. I fear we don't have that kind of discipline here, for instance enforcing clear and descriptive Commit comments is not something I see happening on this project. Sure we could request it as part of the process, but it would only mean more work on our side to check that those requirements are met. We have guidelines in place for code and asset submission, but they are often ignored. |
There is a lot of features currently in development, and they need to be properly merged.
One of the ways to mitigate the merges is to set up proper tools for the machine itself.
You can have a small collection of dotfiles, like .gitattributes, .vs, and such to set them up.
It is even better if there will be a bash script to run and check if everything is set-up correctly
Take a look into the VRTK 4 - Tillia/Zillia or a newly disclosed google's Tilt Brush.
In general, when people use proper good tools, they will be more productive 👌
👉By the way, it can even include the list of packages to use within the IDE itself.
Rider/VS Community with Resharper/Code snippets, Shaderlab Support, VS code with a few required packages installed, etc.
👀 Please take a look into Microsoft's Code of conduct and related features -
https://github.com/microsoft/MixedRealityToolkit-Unity
The text was updated successfully, but these errors were encountered: