Skip to content

fix(ci): prevent overwriting report artifacts#979

Merged
matejchalk merged 1 commit intomainfrom
fix-overwritten-artifacts
Mar 11, 2025
Merged

fix(ci): prevent overwriting report artifacts#979
matejchalk merged 1 commit intomainfrom
fix-overwritten-artifacts

Conversation

@matejchalk
Copy link
Collaborator

@matejchalk matejchalk commented Mar 11, 2025

Fixes #946

All output files are now immediately copied to a special .code-pushup/.ci folder, to prevent command outputs conflicting with artifact files.

CI output files structure

standalone - push

.code-pushup/
├─ .ci/
│  ├─ .current/
│  │  ├─ report.json
│  │  ├─ report.md

standalone - pull request

.code-pushup/
├─ .ci/
│  ├─ .comparison/
│  │  ├─ report-diff.json
│  │  ├─ report-diff.md
│  ├─ .current/
│  │  ├─ report.json
│  │  ├─ report.md
│  ├─ .previous/
│  │  ├─ report.json
│  │  ├─ report.md # if cache not used

monorepo - push

.code-pushup/
├─ .ci/
│  ├─ <project-name>/
│  │  ├─ .current/
│  │  │  ├─ report.json
│  │  │  ├─ report.md
│  ├─ # ... other projects ...

monorepo - pull request

.code-pushup/
├─ .ci/
│  ├─ <project-name>/
│  │  ├─ .comparison/
│  │  │  ├─ report-diff.json
│  │  │  ├─ report-diff.md
│  │  ├─ .current/
│  │  │  ├─ report.json
│  │  │  ├─ report.md
│  │  ├─ .previous/
│  │  │  ├─ report.json
│  │  │  ├─ report.md # if cache not used
│  ├─ # ... other projects ...
│  ├─ .comparison/
│  │  ├─ report-diff.md

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 bug something isn't working 🧩 ci 📖 Project documentation improvements or additions to the project documentation 🔬 testing writing tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Report artifacts overwritten in CI

2 participants