<p>Who is responsible for checking that people don't check in commits that have large files? The same people who are responsible for checking that the commits aren't bad in other ways: everyone.</p>

<p>If git is new in your company, make sure that the git education includes this sort of thing. After that, it's up to code reviewers to notice mistakes and correct them.</p>

<p>If it becomes a repeated problem, then whoever is in charge of developer tooling can write some checks that run every night or whatever and let people know that something slipped through. But that seems like overkill in most situations.</p>

CC BY-SA 3.0
2017-06-10T17:25:01.213Z
2017-06-10T17:25:01.213Z
Xiong Chiamiov
<p>Having spent lots of time on github and various private corporate repos on gitlab and such there are certain things that make a huge difference in how good a repo seems to me:</p>

<ul>
<li>Does the <code>README</code> clearly state what the software does?  Hopefully it also provides some code snippets, usage examples, and links to the preferred support forum.  Any build or runtime prerequisites should be clearly spelled out.  Are the primary authors clearly identified?</li>
<li>Are there releases?  If nobody has tagged a release it is easy to assume that the repo is still in early development and not ready for secondary consumption.</li>
<li>Is there a <code>LICENSE</code>?  If I don't know that I can <em>not</em> legally use this software and it makes it much harder to decide to invest the time into it.  On a private corporate server I would assume that everything is valid for internal use, but in some cases that might not work either.</li>
<li>Is there a link to a CI environment?  If the repository doesn't even show the minimal effort at providing tests and access to a CI environment that regularly runs those tests it is a strong vote against it.</li>
<li>Are the build instructions for a new contributor clear?  Some people may have this containerized, but the build instructions for doing it manually should be viable as well.</li>
<li>In a corporate environment I would expect that you could impose other restrictions semi-automatically.  Does the code fit the code formatting guidelines?  Are binaries excluded (a good idea if you have a central artifact repository for such things)?  I wouldn't set an arbitrary size limitation.  Some projects may just grow that large.  That's not great from an organizational perspective, but it is easy enough to implement technically.</li>
</ul>

<p>As for "who" does this I'd say it ends up being a combination of people.  To start you need buy-in from engineering management.  Then it could be a DevOps person or some random developer who implements it in a particular place.</p>

CC BY-SA 3.0
2017-06-11T02:47:58.453Z
2017-06-14T13:40:26.917Z
2017-06-14T13:40:26.917Z
chicks
<p>If you manage a lot of Git repositories (or other, but Git is a specific and common technology), it could happen that depending on users there will be quite different usage profiles. On the worst case end you could face issues with broken repositories (unplanned work, risks) and wasted storage (which is claimed to be cheap but is still accounted for).</p>

<p>Who is responsible for imposing minimum acceptance criteria for "quality of usage" (I cannot find better term for now) and what are realistic quantifiable measures you could install and link to actionables?</p>

<p>Examples of anti-patterns and related risks you might be dealing with:</p>

<ul>
<li><p>Folks upload files, also binaries, which are neither part of the software nor test documents</p></li>
<li><p>Binary libraries instead of using dependency management</p></li>
<li><p>Repos become too big (over 15G as stated here [1])</p></li>
<li><p>Repos are below 15G but the history is many gigabytes</p></li>
<li><p>Other?</p></li>
</ul>

<p>Or, should anybody care about these things at all? (formal objections, not opinions, why not care)</p>

<p>[1] <a href="https://www.quora.com/What-is-the-practical-maximum-size-of-a-Git-repository-full-of-text-based-data" rel="nofollow noreferrer">https://www.quora.com/What-is-the-practical-maximum-size-of-a-Git-repository-full-of-text-based-data</a></p>

<git><scm><risk>
What are feasible quality criteria for Git repositories?
Ta Mu
# What are feasible quality criteria for Git repositories?

(Asked by: username_0 on 2017-06-10)

If you manage a lot of Git repositories (or other, but Git is a specific and common technology), it could happen that depending on users there will be quite different usage profiles. On the worst case end you could face issues with broken repositories (unplanned work, risks) and wasted storage (which is claimed to be cheap but is still accounted for).

Who is responsible for imposing minimum acceptance criteria for "quality of usage" (I cannot find better term for now) and what are realistic quantifiable measures you could install and link to actionables?

Examples of anti-patterns and related risks you might be dealing with:

* Folks upload files, also binaries, which are neither part of the software nor test documents
* Binary libraries instead of using dependency management
* Repos become too big (over 15G as stated here [1])
* Repos are below 15G but the history is many gigabytes
* Other?

Or, should anybody care about these things at all? (formal objections, not opinions, why not care)

[1] <https://www.quora.com/What-is-the-practical-maximum-size-of-a-Git-repository-full-of-text-based-data>

**Tags:** git, scm, risk


---

## 2 answers

### Answer by username_1 on 2017-06-10. Score: 5

Who is responsible for checking that people don't check in commits that have large files? The same people who are responsible for checking that the commits aren't bad in other ways: everyone.

If git is new in your company, make sure that the git education includes this sort of thing. After that, it's up to code reviewers to notice mistakes and correct them.

If it becomes a repeated problem, then whoever is in charge of developer tooling can write some checks that run every night or whatever and let people know that something slipped through. But that seems like overkill in most situations.


---

### Answer by username_2 on 2017-06-11. Score: 2

Having spent lots of time on github and various private corporate repos on gitlab and such there are certain things that make a huge difference in how good a repo seems to me:

* Does the `README` clearly state what the software does? Hopefully it also provides some code snippets, usage examples, and links to the preferred support forum. Any build or runtime prerequisites should be clearly spelled out. Are the primary authors clearly identified?
* Are there releases? If nobody has tagged a release it is easy to assume that the repo is still in early development and not ready for secondary consumption.
* Is there a `LICENSE`? If I don't know that I can *not* legally use this software and it makes it much harder to decide to invest the time into it. On a private corporate server I would assume that everything is valid for internal use, but in some cases that might not work either.
* Is there a link to a CI environment? If the repository doesn't even show the minimal effort at providing tests and access to a CI environment that regularly runs those tests it is a strong vote against it.
* Are the build instructions for a new contributor clear? Some people may have this containerized, but the build instructions for doing it manually should be viable as well.
* In a corporate environment I would expect that you could impose other restrictions semi-automatically. Does the code fit the code formatting guidelines? Are binaries excluded (a good idea if you have a central artifact repository for such things)? I wouldn't set an arbitrary size limitation. Some projects may just grow that large. That's not great from an organizational perspective, but it is easy enough to implement technically.

As for "who" does this I'd say it ends up being a combination of people. To start you need buy-in from engineering management. Then it could be a DevOps person or some random developer who implements it in a particular place.


---

