Skip to content

fix: correct infrastructure bridge - CI/CD, Dockerfile, tests, code quality#8

Merged
MiltonJ23 merged 2 commits intomainfrom
copilot/correct-infrastructure-bridge
Apr 15, 2026
Merged

fix: correct infrastructure bridge - CI/CD, Dockerfile, tests, code quality#8
MiltonJ23 merged 2 commits intomainfrom
copilot/correct-infrastructure-bridge

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 15, 2026

  • Analyze CI/CD failure logs (root cause: go.sum in .gitignore, wiki repo missing)
  • Review all CodeRabbit and Copilot review comments on PR Feat/milestone3 part 1 infrastructure bridge #5
  • Fix .gitignore: remove go.sum, add coverage.out and .env.*
  • Generate and commit go.sum via go mod tidy
  • Replace ci.yml with proper CI workflow (vet + test + build)
  • Add release.yml workflow for Docker image builds on tags
  • Add deploy.yml workflow for Kubernetes deployments with input validation
  • Fix Dockerfile for multi-arch support (use TARGETARCH)
  • Add .dockerignore to exclude dev files from Docker context
  • Fix wiki.yml to handle missing wiki gracefully (continue-on-error)
  • Fix HandlePrice to return 404 for not-found/not-configured errors
  • Fix ConsumeJob to validate maxConcurrency >= 1
  • Fix semaphore re-acquire to observe context cancellation
  • Fix ERP pricing test error message expectations
  • Fix mock transport to respect context cancellation
  • Verify go vet, all tests pass, and build succeeds locally
  • Merge main and verify no regressions
  • Run parallel validation (code review + CodeQL: both clean)
  • Create PR fix: correct infrastructure bridge - CI/CD, Dockerfile, tests, code quality #8 to merge fixes into main (CI awaiting maintainer approval)

Copilot AI and others added 2 commits April 14, 2026 23:59
…-arch Dockerfile support

- Remove go.sum from .gitignore and commit generated go.sum
- Add coverage.out and .env.* to .gitignore
- Replace Build and Push to GHCR workflow with proper CI workflow (vet + test + build)
- Add release.yml workflow for Docker image builds on tags
- Add deploy.yml workflow for Kubernetes deployments with input validation
- Fix wiki.yml with continue-on-error for missing wiki repo
- Fix Dockerfile for multi-arch support (TARGETARCH/TARGETOS build args)
- Add .dockerignore to exclude dev files from Docker context
- Fix HandlePrice to return 404 for not-found/not-configured errors
- Fix ConsumeJob to validate maxConcurrency >= 1
- Fix semaphore re-acquire to observe context cancellation
- Fix ERP pricing test expectations to match current error format
- Fix mock transport to respect context cancellation

Agent-Logs-Url: https://github.com/MiltonJ23/Kliops/sessions/e774138a-ab5c-4263-9a54-59954a716088

Co-authored-by: MiltonJ23 <151161809+MiltonJ23@users.noreply.github.com>
…structure-bridge

Co-authored-by: MiltonJ23 <151161809+MiltonJ23@users.noreply.github.com>
Copilot AI requested a review from MiltonJ23 April 15, 2026 00:07
@MiltonJ23 MiltonJ23 marked this pull request as ready for review April 15, 2026 00:09
@MiltonJ23 MiltonJ23 merged commit 905a30e into main Apr 15, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants