didericis/bot-bottle
1
0
Fork 0
Code Issues 5 Pull Requests 3 Actions Packages Projects Releases Wiki Activity

Unify identity/provisioned_key into key block #235

Merged
didericis merged 5 commits from refactor-key-block into main 2026-06-19 18:31:10 -04:00
Conversation 7 Commits 5 Files Changed 11
+23 -23
1 changed files with 23 additions and 23 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 7a124d7d25 - Show all commits
bot_bottle/manifest_git.py
+16 -16
View File
@@ -195,22 +195,7 @@ def _parse_key_config(
f"allowed: {', '.join(sorted(_KEY_PROVIDERS))}"
)
if provider == "static":
for k in d:
if k not in {"provider", "path"}:
raise ManifestError(
f"bottle '{bottle_name}' {label}.key has unknown key {k!r} "
f"for provider 'static'; allowed: provider, path"
)
path = d.get("path")
if not isinstance(path, str) or not path:
raise ManifestError(
f"bottle '{bottle_name}' {label}.key missing required "
f"string field 'path' for provider 'static'"
)
return ManifestKeyConfig(provider=provider, path=path)
# provider == "gitea"
if provider == "gitea":
didericis marked this conversation as resolved Outdated
didericis commented 2026-06-19 18:20:09 -04:00
Outdated
Review
Copy Link
Copy Source

static should be the default provider, not gitea

static should be the default provider, not gitea
for k in d:
if k not in {"provider", "forge_token_env", "api_url"}:
raise ManifestError(
@@ -234,6 +219,21 @@ def _parse_key_config(
api_url=api_url_raw,
)
# provider == "static"
for k in d:
if k not in {"provider", "path"}:
raise ManifestError(
f"bottle '{bottle_name}' {label}.key has unknown key {k!r} "
f"for provider 'static'; allowed: provider, path"
)
path = d.get("path")
if not isinstance(path, str) or not path:
raise ManifestError(
f"bottle '{bottle_name}' {label}.key missing required "
f"string field 'path' for provider 'static'"
)
return ManifestKeyConfig(provider=provider, path=path)
@dataclass(frozen=True)
class ManifestGitUser: