fixup! Refactor persistence handling to use a factory pattern for bet…

…ter flexibility and add local persistence option

Implement method overrides for persist_new_result in S3Persistence and LocalPersistence classes

benchmarks/persistence/persistence.py

@@ -9,6 +9,7 @@
 from attr import define, field
 from attrs.validators import instance_of
 from git import Repo
+from typing_extensions import override
 
 from benchmarks import Benchmark, Result
 
@@ -70,6 +71,7 @@ def _default_workflow_id(self) -> str:
             raise ValueError("The environment variable GITHUB_RUN_ID is not set.")
         return os.environ["GITHUB_RUN_ID"]
 
+    @override
     def persist_new_result(self, result: Result, benchmark: Benchmark) -> None:
         """Store the result of a benchmark.
 
@@ -126,6 +128,7 @@ def _build_filename(self, result: Result) -> str:
             + f"{metadata.commit_hash}_results.json"
         )
 
+    @override
     def persist_new_result(self, result: Result, benchmark: Benchmark) -> None:
         """Store the result of a benchmark."""
         filename = self._build_filename(result)