Skip to content

feat: generate .meta/prefixes.txt for Maven repository versions#409

Draft
CryptoRodeo wants to merge 12 commits into
pulp:mainfrom
CryptoRodeo:pulp-2004
Draft

feat: generate .meta/prefixes.txt for Maven repository versions#409
CryptoRodeo wants to merge 12 commits into
pulp:mainfrom
CryptoRodeo:pulp-2004

Conversation

@CryptoRodeo

@CryptoRodeo CryptoRodeo commented Jul 9, 2026 •
edited
Loading

Copy link
Copy Markdown

🚧 Work In Progress 👷

Related to #399

📜 Checklist

  • Commits are cleanly separated with meaningful messages (simple features and bug fixes should be squashed to one commit)
  • A changelog entry or entries has been added for any significant changes
  • Follows the Pulp policy on AI Usage
  • (For new features) - User documentation and test coverage has been added

See: Pull Request Walkthrough

@CryptoRodeo
feat: add _compute_prefix and save_prefixes_txt helpers
c12fe24 
- compute two path segement prefixes
- add helper to create and save prefixes file

Signed-off-by: Bryan Ramos <bramos@redhat.com>
@CryptoRodeo
feat: generate .meta/prefixes.txt in _generate_metadata
2c37f2b 
Signed-off-by: Bryan Ramos <bramos@redhat.com>
CryptoRodeo added 10 commits July 9, 2026 16:16
@CryptoRodeo
feat: generate .meta/prefixes.txt in repair_metadata
e66783e 
Signed-off-by: Bryan Ramos <bramos@redhat.com>
@CryptoRodeo
chore: add clearer comments and var names
dc1f3db 
Signed-off-by: Bryan Ramos <bramos@redhat.com>
@CryptoRodeo
chore: lint fixes
8963f34 
Signed-off-by: Bryan Ramos <bramos@redhat.com>
@CryptoRodeo
fix: typo
b56bf53 
value_list -> values_list

other:
- add prefixes.txt header
- rename variable to match others in the same function

Signed-off-by: Bryan Ramos <bramos@redhat.com>
@CryptoRodeo
test: adding artifacts generates .meta/prefixes.txt
119c699 
Signed-off-by: Bryan Ramos <bramos@redhat.com>
@CryptoRodeo
test: adding artifact with new prefix updates prefix file
629bc82 
Signed-off-by: Bryan Ramos <bramos@redhat.com>
@CryptoRodeo
test: Removing the last artifact with a prefix updates prefixes.txt
35d85fc 
Signed-off-by: Bryan Ramos <bramos@redhat.com>
@CryptoRodeo
test: Adding an artifact within an existing prefix does NOT regenerat…
f24b9d9 
…e prefixes.txt

Signed-off-by: Bryan Ramos <bramos@redhat.com>
@CryptoRodeo
test: The .meta/prefixes.txt file is downloadable from the content app.
7fa5255 
also update another test.

Signed-off-by: Bryan Ramos <bramos@redhat.com>
@CryptoRodeo
fix: update existing tests
098e7b7 
new prefixes.txt file messes with previous counts.

Signed-off-by: Bryan Ramos <bramos@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

multi-commit no-changelog no-issue

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@CryptoRodeo