Allow publication of empty repositories#1424
Conversation
9c23ff2 to
e3ad64c
Compare
e3ad64c to
3d5da69
Compare
582c995 to
ce0c6a6
Compare
quba42
left a comment
There was a problem hiding this comment.
Things I spent some time considering;
- Naming of the new settings.
- Can the new indentation level in the publish function be avoided?
- Can something be done against the spaghetti style taking over publish?
My conclusion: I could not come up with any improvements that are not out of scope for this change.
I did find and fixed one more edge case: If both simple and structured is enabled, we should just rely on the simple metadata (which is always published) and not also publish fallback structured empty metadata. This also prevents failure/collision for the case STRUCTURED_EMPTY_REPO_DISTRIBUTION = "default". I then wen't and made that the new default so our new "default empty structured metadata" is more like our "default simple mode metadata" for consistency. It is a setting so users who don't like this can change it.
Backport to 3.8: 💚 backport PR created✅ Backport PR branch: Backported as #1455 🤖 @patchback |
Backport to 3.8: 💔 cherry-picking failed — could not push❌ Current GitHub App installation does not grant sufficient privileges for pushing to https://github.com/pulp/pulp_deb.git. Lacking the underlying command output was: $ git --git-dir /tmp/pulp--pulp_deb---3.8---4nycmxoh---PR-1424.git/.git --work-tree /tmp/pulp--pulp_deb---3.8---4nycmxoh---PR-1424.git -c user.email=45432694+patchback[bot]@users.noreply.github.com -c user.name=patchback[bot] -c diff.algorithm=histogram push --force-with-lease origin HEAD
[RETURN CODE]: 1
[OUTPUT]:
[STDERR]:
Backporting merged PR #1424 into main
🤖 @patchback |
Allow publication of empty repositories (cherry picked from commit 482bd0d)
No description provided.