Skip to content

Conversation

@matheusandre1
Copy link
Contributor

#80

@matheusandre1
Copy link
Contributor Author

Add the user secrets to the main repository.

secrets.QUAY_USERNAME
secrets.QUAY_PASSWORD

Copy link
Owner

@mcruzdev mcruzdev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@matheusandre1 I left a simple comment, I think for whatsapp we can keep as is.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds OCI image build and publish capabilities for the timeless-api (Java) and whatsapp (Node.js) applications using Cloud Native Buildpacks. The implementation uses Paketo buildpacks to create container images and publishes them to Quay.io registry.

Key changes:

  • Integration of Pack CLI and Quay.io registry publishing into the CI pipeline for both applications
  • Addition of Aptfile for specifying system dependencies required by the WhatsApp application (Chromium and related libraries)
  • Creation of a Dockerfile for the WhatsApp application (though it won't be used with the buildpacks approach)

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 7 comments.

File Description
.github/workflows/pipeline.yml Added Pack CLI setup, Quay.io authentication, and image build/publish steps for both timeless-api and whatsapp jobs
whatsapp/Dockerfile Added Dockerfile with Node.js base image and Puppeteer/Chromium dependencies (not used by buildpacks)
whatsapp/Aptfile Added system package dependencies for Chromium browser support required by whatsapp-web.js

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@matheusandre1
Copy link
Contributor Author

cc: @mcruzdev , Please advise me on possible pipeline improvements.

@matheusandre1
Copy link
Contributor Author

I tried following this

Copy link
Owner

@mcruzdev mcruzdev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot!

Let's see if it works!

@mcruzdev mcruzdev merged commit 8d4d254 into mcruzdev:main Dec 13, 2025
3 checks passed
@matheusandre1 matheusandre1 deleted the ci-issue-80 branch January 5, 2026 14:38
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