Skip to content

Add artifact upload and download steps for GeoLite2 City database in … - #8

Merged
youssef-ar merged 1 commit into
mainfrom
feat/cicd
Apr 3, 2026
Merged

youssef-ar merged 1 commit into
mainfrom
feat/cicd

Conversation

@youssef-ar

@youssef-ar youssef-ar commented Apr 3, 2026 •

Copy link
Copy Markdown
Contributor

…CI workflow

Summary by CodeRabbit

  • Chores
    • Improved CI/CD pipeline artifact handling for build dependencies.

@coderabbitai

coderabbitai Bot commented Apr 3, 2026 •

Copy link
Copy Markdown
ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Free

Run ID: 76ef9589-821a-43b2-b986-baf86258dd04

📥 Commits

Reviewing files that changed from the base of the PR and between be41f7c and f6e2fa9.

📒 Files selected for processing (1)
  • .github/workflows/ci.yml

📝 Walkthrough

Walkthrough

The pull request adds artifact management to the GitHub Actions CI workflow. A new step uploads the data/GeoLite2-City.mmdb file as an artifact in the test job, and the docker-build job downloads this artifact to the data directory before building the Docker image.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow
.github/workflows/ci.yml
Added upload step for GeoLite2-City.mmdb artifact in test job with failure enabled when file is missing. Added corresponding download step in docker-build job to retrieve artifact before Docker image construction.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~4 minutes

Poem

🐰 A workflow so fine, with artifacts to share,
Data hops between jobs with algorithmic care,
GeoLite flows where Docker builds await,
Each job passes the torch—how elegant and great!


Note

🎁 Summarized by CodeRabbit Free

The PR author is not assigned a seat. To perform a comprehensive line-by-line review, please assign a seat to the pull request author through the subscription management page by visiting https://app.coderabbit.ai/login.

Comment @coderabbitai help to get the list of available commands and usage tips.

@sonarqubecloud

sonarqubecloud Bot commented Apr 3, 2026

Copy link
Copy Markdown

@youssef-ar
youssef-ar merged commit 54af099 into main Apr 3, 2026
5 checks 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.

1 participant