RepositoryConfig: Instance should not be changed once populated

Once a RepositoryConfig instance gets populated, it must never be
changed again. Therefore, all functions accepting these instances
should only take them as pointers to const.

1 files changed, 2 insertions, 2 deletions

diff --git a/src/buildtool/execution_api/local/local_api.hpp b/src/buildtool/execution_api/local/local_api.hpp
index fb76237d..f69a5be2 100644
--- a/src/buildtool/execution_api/local/local_api.hpp
+++ b/src/buildtool/execution_api/local/local_api.hpp
@@ -47,7 +47,7 @@
 /// \brief API for local execution.
 class LocalApi final : public IExecutionApi {
   public:
-    explicit LocalApi(std::optional<gsl::not_null<RepositoryConfig*>>
+    explicit LocalApi(std::optional<gsl::not_null<const RepositoryConfig*>>
                           repo_config = std::nullopt)
         : repo_config_{std::move(repo_config)} {}
 
@@ -533,7 +533,7 @@ class LocalApi final : public IExecutionApi {
     }
 
   private:
-    std::optional<gsl::not_null<RepositoryConfig*>> repo_config_{};
+    std::optional<gsl::not_null<const RepositoryConfig*>> repo_config_{};
     gsl::not_null<Storage const*> storage_ = &Storage::Instance();
 };