Does pro grammatically check-in the best practice?

Does pro grammatically check-in the best practice?

From time to time I receive request to add check-in step to build definition. In general this is simple operation: some step change file(s), and customer need that this files put to source control.
But is it good scenario, and what a risk will you receive after this implementation? Let's discuss it in this article.