sarman

sarman pushed to feat/pr-review-workflow at sarman/tftsr-devops_investigation

2026-04-13 01:07:23 +00:00

30d26ed0c1 fix(ci): address second AI review — || true, ca-certs, cache@v4, key suffixes

sarman commented on pull request sarman/tftsr-devops_investigation#36

2026-04-13 01:03:11 +00:00

perf(ci): pre-baked images + cargo/npm caching (~70% faster builds)

🤖 Automated PR Review:\n\nHere's a comprehensive review of the PR changes, covering correctness, security, and best practices:

sarman pushed to feat/pr-review-workflow at sarman/tftsr-devops_investigation

2026-04-13 00:59:44 +00:00

397cef383d fix(ci): resolve test.yml failures — Cargo.lock, updated test assertions

sarman commented on pull request sarman/tftsr-devops_investigation#36

2026-04-13 00:46:00 +00:00

perf(ci): pre-baked images + cargo/npm caching (~70% faster builds)

🤖 Automated PR Review:\n\n### Code Review: Pre-baked Images + Cargo/npm Caching

sarman pushed to feat/pr-review-workflow at sarman/tftsr-devops_investigation

2026-04-13 00:42:12 +00:00

84dd6d57c3 fix(docker): add ca-certificates to arm64 base image step 1

sarman commented on pull request sarman/tftsr-devops_investigation#36

2026-04-12 23:57:57 +00:00

perf(ci): pre-baked images + cargo/npm caching (~70% faster builds)

🤖 Automated PR Review:\n\nHere's a comprehensive review of the PR changes, covering correctness, security, and best practices:

sarman pushed to feat/pr-review-workflow at sarman/tftsr-devops_investigation

2026-04-12 23:55:01 +00:00

7323c93463 docs(docker): expand rebuild trigger comments to include OpenSSL and Tauri CLI

sarman commented on pull request sarman/tftsr-devops_investigation#36

2026-04-12 23:34:47 +00:00

perf(ci): pre-baked images + cargo/npm caching (~70% faster builds)

🤖 Automated PR Review:\n\nHere's a comprehensive review of the PR changes, covering correctness, security, and best practices:

sarman commented on pull request sarman/tftsr-devops_investigation#36

2026-04-12 23:34:02 +00:00

perf(ci): pre-baked images + cargo/npm caching (~70% faster builds)

🤖 Automated PR Review:\n\nThis PR introduces significant performance improvements by leveraging pre-baked Docker images and caching, aiming for ~70% faster builds. Below is a detailed review covering correctness, security, and best practices, with actionable suggestions.

sarman pushed to feat/pr-review-workflow at sarman/tftsr-devops_investigation

2026-04-12 23:30:22 +00:00

1d0f420042 fix(ci): replace docker:24-cli with alpine + docker-cli in build-images

sarman pushed to feat/pr-review-workflow at sarman/tftsr-devops_investigation

2026-04-12 23:29:43 +00:00

f59559327e fix(ci): address AI review — rustup idempotency and cargo --locked

sarman released TFTSR v0.2.50 at sarman/tftsr-devops_investigation

2026-04-12 23:23:03 +00:00

sarman commented on pull request sarman/tftsr-devops_investigation#36

2026-04-12 23:22:01 +00:00

perf(ci): pre-baked images + cargo/npm caching (~70% faster builds)

🤖 Automated PR Review:\n\n### Code Review: Pre-baked Images + Cargo/npm Caching

sarman created pull request sarman/tftsr-devops_investigation#36

2026-04-12 23:18:37 +00:00

perf(ci): pre-baked images + cargo/npm caching (~70% faster builds)

sarman pushed to feat/pr-review-workflow at sarman/tftsr-devops_investigation

2026-04-12 23:17:39 +00:00

f1461a0d7e perf(ci): use pre-baked images and add cargo/npm caching

sarman pushed tag v0.2.50 to sarman/tftsr-devops_investigation

2026-04-12 23:08:59 +00:00

sarman pushed to master at sarman/tftsr-devops_investigation

2026-04-12 23:08:48 +00:00

4fa01ae7ed Merge pull request 'feat/pr-review-workflow' (#35) from feat/pr-review-workflow into master

181b9ef734 fix: harden pr-review workflow — secret redaction, log safety, auth header

144a4551f2 fix: revert to two-dot diff — three-dot requires merge base unavailable in shallow clone

47b2e824e0 fix: replace github.server_url with hardcoded gogs.tftsr.com for container access

82aae00858 fix: resolve AI review false positives and address high/medium issues

Compare 20 commits »

sarman merged pull request sarman/tftsr-devops_investigation#35

2026-04-12 23:08:48 +00:00

feat/pr-review-workflow

sarman commented on pull request sarman/tftsr-devops_investigation#35

2026-04-12 23:07:10 +00:00

feat/pr-review-workflow

🤖 Automated PR Review:\n\nThis PR introduces a new GitHub Actions workflow for automated PR reviews using Ollama, along with version bumps in package.json and src-tauri/Cargo.toml/tauri.conf.json. Below is a detailed analysis covering correctness, security, and best practices.

sarman pushed to feat/pr-review-workflow at sarman/tftsr-devops_investigation

2026-04-12 23:03:25 +00:00

181b9ef734 fix: harden pr-review workflow — secret redaction, log safety, auth header