3 ms·
> The entire system also runs locally. I think approaches like this don't need to run other than locally. Maybe integrated as pre-push hook. The system is nond
by rzmmm 5mo ago
> The entire system also runs locally.
I think approaches like this don't need to run other than locally. Maybe integrated as pre-push hook. The system is nondeterministic, so it's at odds with the purpose of CI.
- new_account_101 5mo ago[dead]
- fhd2 5mo agoI'd argue it's pretty much like monitoring, which certainly benefits from multiple people seeing the same stats and alerts. I agree it's at odds with CI/CD and should probably not block anything, like deterministic checks commonly do.
- proofofcontempt 5mo agoI'm not sure the people integrating it into CI process understand what CI is.
- Scea91 5mo agoSame can be said about human review if the argument is non-determinism.
- proofofcontempt 5mo agoHuman review is about learning and there's an implied social contract in that someone is giving you their time to make you better. It isn't necessarily necessary but replacing it with AI shows a fundamental misunderstanding of why it is part of the process.
- krzyk 5mo agoIt is starting the review during CI (CI just triggers the review), not blocking merges like failed build or lint failures.
- rzmmm 5mo ago[dead]