isomerpages / isomercms-backend

A static website builder and host for the Singapore Government
5 stars 1 forks source link

fix(repoChecker): unintended alarms #1176

Closed kishore03109 closed 7 months ago

kishore03109 commented 7 months ago

Problem

due to logging of the simple git operation, this triggered other alarms. since this is not a critical flow, fail silently.

also, running the check using the form in parallel to reduce the number of errors that has occured (when i tried running this in prod yesterday, the git.lock file kept perventing the commits from occuring. this is a problem since the form was used for observability, and this would lead to the reports not being committed to remote)

Tests

Screenshot 2024-02-29 at 8 59 26 AM

mergify[bot] commented 7 months ago

⚠️ The sha of the head commit of this PR conflicts with #1177. Mergify cannot evaluate rules on this PR. ⚠️