[macalbert]

Summary

The Python SDK changelog listed the Azure Key Vault / fix (#623) under 0.6.0, but the sdk-python/v0.6.0 tag points to #624, which predates #623, so 0.6.0 on PyPI does not include it. The fix first shipped in 1.0.0, so the entry moves there.

Changes

• docs(changelog): move the "Azure Key Vault references with / resolve the same secret as the CLI" entry from [0.6.0] to [1.0.0] under ### Fixed in docs/changelogs/sdk-python.md

Testing

• pnpm test passes (N/A: documentation only)
• pnpm lint passes (N/A: documentation only)
• Manual verification: git merge-base --is-ancestor 876bd8fd sdk-python/v0.6.0 fails and succeeds for sdk-python/v1.0.0

Related

#623, #624

Summary by CodeRabbit

• Documentation
  • Updated the Python SDK changelog to list the Azure Key Vault reference behavior under version 1.0.0 instead of 0.6.0. The entry notes that references replace / with - and ignore leading slashes; other values are sent unchanged.

[coderabbitai]

No actionable comments were generated in the recent review.

Recent review info

Run configuration

• Source mode: Auto
• Repository: macalbert/envilder
• Branch: macalbert/python-changelog-azure-fix
• File: .coderabbit.yaml
• Commit: 3aaeda5
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: 4160b545-f812-4b9e-bd9d-e8130c55ce8d

Commits

Reviewing files that changed from the base of the PR and between 83f4fb3 and 3aaeda5.

Files selected for processing: 1

• docs/changelogs/sdk-python.md

Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 0 remain after this review.

Walkthrough

The Python SDK changelog moves the Azure Key Vault reference-normalization entry from version 0.6.0 to version 1.0.0.

Changes: Python SDK changelog

Layer / File(s)	Summary
Release entry placement docs/changelogs/sdk-python.md	The 1.0.0 Fixed section adds the entry describing reference normalization. The 0.6.0 Fixed section removes the entry.

Priority: Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Possibly related PRs

• macalbert/envilder#623: Implements the Azure Key Vault rule described by this changelog entry.
• macalbert/envilder#612: Documents the prior .NET SDK behavior for Azure Key Vault secret names.

Merge Risk: Minimal · up to 3aaed

The changelog correction does not affect runtime or deployment behavior.

Pre-merge checks: 7

Check name	Status	Explanation
Title check		The title follows Conventional Commits: type is docs, scope is changelog, and the subject is lowercase. It is 63 characters, under the 100-character limit. It names the Azure path fix and the Python S…
Description check		The description follows the repository template. It has a clear summary of why the entry moves, a changes list, and a testing section. It also links related issues #623 and #624. The template's 'What …
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.
Architecture Boundaries		The pull request changes only docs/changelogs/sdk-python.md. No changed file is under src/envilder/core/domain/ or src/envilder/core/application/, so the specified import-boundary conditions do …
Tests Accompany Code Changes		The pull request changes only docs/changelogs/sdk-python.md. It changes no non-test source file under src/envilder/ or src/sdks/, so the check does not apply.
Release Coherence		No version field changed in the reviewed pull request. The only changed file is docs/changelogs/sdk-python.md, so the check's version-change condition does not apply.

Full details: Title check

Explanation

The title follows Conventional Commits: type is docs, scope is changelog, and the subject is lowercase. It is 63 characters, under the 100-character limit. It names the Azure path fix and the Python SDK 1.0.0 section, which matches the changeset that moves the entry from 0.6.0 to 1.0.0.

Full details: Description check

Explanation

The description follows the repository template. It has a clear summary of why the entry moves, a changes list, and a testing section. It also links related issues #623 and #624. The template's 'What does this PR do?', 'Related issues', 'Type of change', and 'Checklist' headings are reworded or missing. The 'Type of change' checkbox is not ticked, and 'Notes for reviewer' is absent. These are minor gaps, so the description is mostly complete.

Full details: Architecture Boundaries

Explanation

The pull request changes only docs/changelogs/sdk-python.md. No changed file is under src/envilder/core/domain/ or src/envilder/core/application/, so the specified import-boundary conditions do not apply.

Finishing Touches

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}