Should we commit generate type files to Git?
# contributing-to-airbyte
j
Should we commit generate type files to Git?
u
tu or td
u
this is for both python and java
u
In general I would say no
u
it should be consistent.
u
Why not?
u
It requires
./gradlew build
to run before IDEs won't complain
u
you never have diffs on generation settings-level changes
u
also for new people looking at the repo they don't see implementations of our entire data model, which is confusing
u
I just see generated code as one more place whre you can break the build
u
The downside is just a little bit more effort for merge commits when there are data model changes? Maybe confusion over if the files are editable?
u
in a way that you don't notice
u
the merge piece is a big deal
u
as well as diff
u
I don't really think so, can just clean and regenerate in the event of the merge commit
u
I would argue that the diff is desirable
u
I am fine if for now for python we commit the generated code because we don't have a great process
u
Especially since we have some custom override logic for Java
u
In the middle of fixing master. We can discuss it later