Remove combined namespace read only git messaging on free user efficiency
The following discussion from !102429 (merged) should be addressed:
-
@dstull started a discussion: (+3 comments) @csouthard @vij @mhamda @sheldonled I want to raise awareness that the combined git messaging for free user cap and storage still exists in the codebase. Here that this code still exists. When growth divested of owning the combined alert here we decided to leave the git messaging piece here for combined alerting in case it could be re-used by your team. I want to make sure you all are aware of it and where
git_check_over_limit!
is called in the git areas of the codebase.We have a few options here once you have time to analyze:
- Remove the combined functionality of this method(i.e. remove this conditional and the blocks being sent to it by lfs)
- Reuse for any plans you all have for combined git messaging.