build(pbn): BPP na paczkach PBN z PyPI (0.2, wave)#601
Open
mpasternak wants to merge 5 commits into
Open
Conversation
mpasternak
changed the title
Atrybut wiążący model w getterach stron PBN niósł nazwę z monorepo (pbn_api). Ujednolicony z pakietem django-pbn-client: baza deklaruje class-attr `model_class = None`, konstruktor nie przesłania go None-em, a InstitutionGetter/ZrodlaGetter/PublisherGetter oraz shim ThreadedMongoDBSaver używają nowej nazwy. manage.py check: OK; pakiet + test_helpers: 40 passed. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…ges/
Krok 1 planu ekstrakcji: BPP konsumuje pbn-client i django-pbn-client jako
normalne zależności z PyPI (0.1.0, github.com/iplweb/*) zamiast lokalnych
memberów uv workspace.
- usunięto katalog packages/ (kopie robocze) — źródłem jest teraz PyPI,
- pyproject: usunięto [tool.uv.workspace] i wpisy `{ workspace = true }`
z [tool.uv.sources] (django-import-export git-source zostaje); deps
pbn-client/django-pbn-client bez zmian,
- pytest.ini: testpaths = src (bez packages),
- Dockerfile: usunięto 4× `COPY packages/` (obraz nie ma już tego katalogu),
- .dockerignore: usunięto regułę README paczek,
- uv.lock: przeliczony (paczki z indeksu, nie editable).
Weryfikacja: paczki zainstalowane z PyPI (==0.1.0, nie editable);
manage.py check OK; makemigrations --check bez dryfu w pbn_api/django_pbn_client
(pozostały dryf to pre-existing third-party AutoField: favicon/flexible_reports/
siteblog); test_helpers + test_validators 31 passed.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
- pbn-client / django-pbn-client: >=0.2,<0.3 (wave 0.2 z PyPI), uv.lock przeliczony, - przepisany docs/deweloper/pakiety-pbn.md: architektura PyPI zamiast workspace (usunięty opis packages/ + Docker copy + komend na nieistniejące ścieżki), - zaktualizowane docs/deweloper/index.md i newsfragment, - usunięte 48 nieaktualnych wpisów packages/* z .test_durations. Weryfikacja: manage.py check OK; makemigrations --check bez dryfu pbn; test_helpers + test_validators 31 passed na paczkach 0.2. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
pbn-client 0.2 przemianował parametr konstruktora publication_pk -> correlation_id (z property publication_pk do odczytu, ale nie jako kwarg konstruktora). Test kolejki konstruował wyjątek z publication_pk= -> TypeError. Adopcja nowej nazwy w teście. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
mpasternak
force-pushed
the
feat/pbn-pypi-rewire
branch
from
4f3405d to
a27cafa
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Przepięcie BPP na reużywalne pakiety PBN publikowane na PyPI, w wersji 0.2 (skoordynowane wydanie wave).
Pakiety (github.com/iplweb/*, PyPI 0.2.0)
pbn-client 0.2.0— protokół PBN + identyfikatory,PublicationNotFound, publiczne helpery oświadczeń,normalize_author_name,RetryPolicy/iter_pages, cleanup API (z aliasami).django-pbn-client 0.2.0—is_deleted,get_or_download,download_to_model; abstrakcyjne modele + pobieranie stron.Zmiana w BPP (net)
packages/+[tool.uv.workspace]/sources, 4×COPY packages/z Dockerfile,pytest.ini/.dockerignore,pbn-client>=0.2,<0.3,django-pbn-client>=0.2,<0.3z PyPI,uv.lockprzeliczony,pbn_client/django_pbn_client(re-eksporty/shimy),docs/deweloper/pakiety-pbn.md(architektura PyPI zamiast workspace) +index.md+ newsfragment,.test_durations(48 wpisów usuniętych testów).Weryfikacja
==0.2.0, nie editable),manage.py check: OK,makemigrations --check: brak dryfu wpbn_api/django_pbn_client(pozostały dryf to pre-existing third-party AutoField),test_helpers+test_validators: 31 passed na 0.2.Zawiera review Codexa krok-1 (rewire technicznie zdrowy; jedyny blocker — nieaktualny przewodnik — naprawiony w tym PR). Sugerowany squash-merge.
🤖 Generated with Claude Code