search

google / usd_from_gltf

Apache License 2.0
550 stars 86 forks source link

Improved commit messages #45

Closed StephenRadachy closed 4 years ago

StephenRadachy commented 4 years ago

👋Truly amazing and useful tool y'all built.

you're awesome

Moving forward, can we have more descriptive commit messages?

Most of the commits on the master branch are Internal change or just generally not representative of what the commit relates to.

It would be awesome to have a bit more context as to why you're making a change instead of having to read through the diff and guess.

Thanks!

hujasonx commented 4 years ago

Thanks for bringing this up. I assumed that this was an unavoidable result of us using a tool to keep an internal repo and this public repo in sync. Turns out (just found out about this like right now), that tool actually has a way to set the public-facing commit message. From now on I'll try to set public commit messages with each code change.

Thanks! Jason