Repository navigation
feature/Text-stats-import-path · L-261016-eaf5d9 - #21
Merged
Merged
Conversation
The step analyze_text names its function text_stats_funcs.text_stats_report, its import path from the package's root, which is what `pipelex fix function-names` writes. A Pipelex release later than 0.86.1 refuses the flat name, and 0.86.1 and earlier refuse the import path, so the change is breaking and the next release is a minor. The changelog records it under [Unreleased]; the release commit sets the version in every manifest, in lockstep. The release rules now count a package that stops running on a Pipelex release its previous tag ran on as breaking. Co-Authored-By: Claude Opus 5.5 <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
text_statsnames its PipeFunc function by import path from the package root,text_stats_funcs.text_stats_report, which pipelex requires from the release after 0.86.1 on. The changelog marks it breaking, so the next library release is v0.2.0, and the release guidance now counts a package that stops running on a Pipelex release the previous tag ran on as breaking.Advances L-261016-eaf5d9
Advances L-261012-989e43
🤖 Generated with Claude Code
Summary by cubic
Renames the
text_statsstep's function to its import path (text_stats_funcs.text_stats_report) so it runs on Pipelex releases after 0.86.1, which require that form and refuse the flat name; Pipelex 0.86.1 and earlier refuse the new name, making the change breaking and the next release a minor (v0.2.0).text_statsat@v0.1.7or earlier on older Pipelex releases.Written for commit 4557a69. Summary will update on new commits.