New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(gha)!: use composite action for semantic release (#692) #741
base: master
Are you sure you want to change the base?
Conversation
@zckv I have moved the changes from #692 over here. I'm thinking I want to add the following to this PR (based off of the breakage in the previous one):
Will make PRs against this branch for each of those, obviously no obligation but if you still want to work on the change would you mind doing the same please? That way it can all be merged at once 🙏 |
Later versions reverted python-semantic-release/python-semantic-release#692 which breaks builds on Python projects requiring a newer interpreter than 3.10, which their container uses. Should be fine to update after python-semantic-release/python-semantic-release#741 is merged again. Signed-off-by: Felix Kaechele <felix@kaechele.ca>
Would really love to see this PR go through because without it, projects that use a newer Python than 3.10 can't use the action at all. Pinning to 8.3.0 for now as well. |
9f8c42c
to
c5b6e58
Compare
Resolves: #692 Co-authored-by: Bernard Cooke <bernard-cooke@hotmail.com>
c5b6e58
to
a5bf98e
Compare
@eikowagenknecht, although this composite action has not been merged yet. #914 has and it bumped the Docker action up to Python:3.12-bookworm as the base environment which will allow building of projects that use a minimum version of |
Hello 👋 Wouldn't it be simpler to create a separate action repository ? This solution would avoid a "breaking" change on this project for something that is not related to the PSR python code. |
No, I don't believe so. It sounds nice up front but we would then have to keep both projects in sync. This action change would still be a breaking change which would then cause a version number drift of the version numbers of the project which likely confuses the configurations used and the users. This may not matter as the proper version is a set dependency in the action. I plan to wrap it in with the next breaking change for the project. The release will have a full migration instructions and such with it. Maybe at the breaking change it's moved to another repository which doesn't change all that much besides the version it points to upon this repository update. |
Thanks, working fine for 3.12 now. |
No description provided.