Skip to content

fix: remove kaniko reference from profile README#19

Closed
labrats-ai-agents[bot] wants to merge 1 commit intomainfrom
fix/15-remove-kaniko-references
Closed

fix: remove kaniko reference from profile README#19
labrats-ai-agents[bot] wants to merge 1 commit intomainfrom
fix/15-remove-kaniko-references

Conversation

@labrats-ai-agents
Copy link
Copy Markdown
Contributor

Summary

  • Removes deprecated Kaniko from the CI/CD row in the technology stack table in profile/README.md
  • Replaces it with Docker Build & Push to reflect the org-wide adoption of docker/build-push-action per ADR-0008

References

Changes

File Change
profile/README.md Replace Kaniko with Docker Build & Push in tech stack table

Replace deprecated kaniko with docker/build-push-action in the
technology stack table. Kaniko has been deprecated org-wide per ADR-0008.

Closes #15
@labrats-ai-agents
Copy link
Copy Markdown
Contributor Author

Reviewed PR #19 as @AI-Docs.

Review Summary: The change correctly removes the deprecated Kaniko reference from the CI/CD technology stack row in profile/README.md and replaces it with Docker Build & Push, aligning with ADR-0008. The change is minimal, accurate, and appropriate.

Proceeding to update organization-level documentation in labrats-work/org.docs.

@labrats-ai-agents
Copy link
Copy Markdown
Contributor Author

labrats-ai-agents Bot commented Mar 22, 2026

Documentation (@ai-docs) — Completed

Job: 536

@labrats-ai-agents
Copy link
Copy Markdown
Contributor Author

🕒 Documentation (@ai-docs) — Picked up

Job: 536

@labrats-ai-agents
Copy link
Copy Markdown
Contributor Author

Documentation review complete (@ai-docs)

The PR feedback has been addressed. To summarize:

  • PR change: Replaces Kaniko with Docker Build & Push in the CI/CD row of the technology stack table in profile/README.md — this is correct and aligns with ADR-0008
  • No code changes required: The existing change on this PR is accurate and ready to merge
  • Org docs: The tracking item in labrats-work/org.docs#28 has been noted; this PR's change reflects the org-wide adoption of docker/build-push-action

This PR is ready for merge. @ai-reviewer may approve when ready.

@labrats-ai-agents
Copy link
Copy Markdown
Contributor Author

labrats-ai-agents Bot commented Mar 22, 2026

Documentation (@ai-docs) — Completed

Job: 541

@labrats-ai-agents
Copy link
Copy Markdown
Contributor Author

Documentation (@ai-docs) — In progress

Job: 541

@tomp736
Copy link
Copy Markdown
Contributor

tomp736 commented Mar 22, 2026

@ai-tom — Please review this PR. Check for correctness, adherence to org standards, and any issues. Approve if it looks good, or request changes with specific feedback.

@labrats-ai-agents
Copy link
Copy Markdown
Contributor Author

labrats-ai-agents Bot commented Mar 22, 2026

Thomas Pisula (@ai-tom) — Completed

Job: 600

@labrats-ai-agents
Copy link
Copy Markdown
Contributor Author

Thomas Pisula (@ai-tom) — In progress

Job: 600

@labrats-ai-agents
Copy link
Copy Markdown
Contributor Author

Reviewed. The intent is correct — removing the Kaniko reference aligns with ADR-0008 and the org-wide switch to docker/build-push-action.

Issue: This PR now conflicts with main. PR #20 already merged the same fix (fix(profile): replace kaniko with docker/build-push-action in README), so the base line "Kaniko" no longer exists in main.

The change in this PR uses "Docker Build & Push" while #20 used "docker/build-push-action" (the action name). Both are valid, but #20 is already merged.

Recommendation: Close this PR as superseded by #20. Issue #15 has been addressed.

@labrats-ai-agents
Copy link
Copy Markdown
Contributor Author

Closing as superseded by #20, which already merged the same fix to main.

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.

Remove kaniko references from profile README

1 participant