Skip to content

Run Black via PDM on CI#21

Merged
tisnik merged 1 commit intolightspeed-core:mainfrom
tisnik:run-black-via-pdm-on-ci
Jun 6, 2025
Merged

Run Black via PDM on CI#21
tisnik merged 1 commit intolightspeed-core:mainfrom
tisnik:run-black-via-pdm-on-ci

Conversation

@tisnik
Copy link
Copy Markdown
Collaborator

@tisnik tisnik commented Jun 6, 2025

Description

Run Black via PDM on CI

Type of change

  • Refactor
  • New feature
  • Bug fix
  • CVE fix
  • Optimization
  • Documentation Update
  • Configuration Update
  • Bump-up service version
  • Bump-up dependent library
  • Bump-up library or tool used for development (does not change the final image)
  • CI configuration change
  • Konflux configuration change
  • Unit tests improvement
  • Integration tests improvement
  • End to end tests improvement

@tisnik tisnik merged commit 2fcad35 into lightspeed-core:main Jun 6, 2025
11 checks passed
romartin added a commit to romartin/rag-content that referenced this pull request Apr 1, 2026
- Use /usr/bin/skopeo absolute path (lightspeed-core#18)
- Add timeout=300 and stderr capture to _fetch_base_archive; wrap
  CalledProcessError and TimeoutExpired with descriptive RuntimeError
  messages including skopeo's stderr (lightspeed-core#19/lightspeed-core#23)
- Remove confusing gzip/diffutils/RUN comment (lightspeed-core#20)
- Streaming refactor: _make_layer_tar writes to a temp file and returns
  sha256 computed by chunked read; _read_docker_archive returns layer
  names only (no layer bytes); build_image_archive streams base layers
  directly from the skopeo archive and custom layer from its temp file,
  keeping peak RAM independent of image size (lightspeed-core#21/lightspeed-core#22)
- Validate --model-dir exists before building image archive; fail fast
  with FileNotFoundError pointing to --exclude-model (lightspeed-core#25)
- Fix corrupted README heading (merge artifact)
- Update tests for all of the above; add test_returns_sha256_hex,
  test_layer_data_not_loaded, test_main_raises_if_model_dir_missing

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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