Skip to content

ci(django-cf): release django-cf from ci - #196

Merged
ryanking13 merged 2 commits into
mainfrom
gyeongjae/django-cf-release-ci
Aug 13, 2026
Merged

ci(django-cf): release django-cf from ci#196
ryanking13 merged 2 commits into
mainfrom
gyeongjae/django-cf-release-ci

Conversation

@ryanking13

Copy link
Copy Markdown
Contributor

Apply the same release process that other packages in this repository use to django-cf, so that the package can be released by commit message.

@ask-bonk

ask-bonk Bot commented Aug 12, 2026

Copy link
Copy Markdown

The artifact naming (distribution-artifacts-django-cf) matches the distribution-artifacts-* glob pattern used in the deploy job. This is correct.

I've completed my thorough review. The changes are well-structured, consistent with the existing release infrastructure for the sibling packages, and the intentional differences (like major_on_zero = false for the 0.x package) are properly documented. The workflow serialization, artifact naming, deploy job integration, PSR configuration, CHANGELOG migration, and cleanup of the old version script are all correct.

LGTM

github run

@dom96 dom96 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this auto release all packages in the repo, even if only one had changes? or is it smart enough to know to only release the packages that changed?

@ryanking13

Copy link
Copy Markdown
Contributor Author

Only the package that the code inside the package directory is modified is released (filtered by path_filters key)

@ryanking13
ryanking13 merged commit fa4b814 into main Aug 13, 2026
21 checks passed
@ryanking13
ryanking13 deleted the gyeongjae/django-cf-release-ci branch August 13, 2026 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants