Skip to content

Conversation

@thedataking
Copy link
Contributor

This PR is a follow up to #1504. It was split off to avoid multiple developers force pushing the same branch.

@thedataking thedataking changed the base branch from master to perl/c2rust-postprocess-split-merge December 15, 2025 00:57
@thedataking thedataking requested a review from kkysen December 15, 2025 00:58
@thedataking thedataking marked this pull request as draft December 15, 2025 00:59
@thedataking thedataking changed the title postprocess: Support multiple transforms and mode postprocess: Support multiple transforms and more Dec 15, 2025
@thedataking thedataking force-pushed the perl/c2rust-postprocess-split-merge-part-two branch 4 times, most recently from e9e8109 to 2a1a0b9 Compare December 15, 2025 08:02
@kkysen kkysen force-pushed the perl/c2rust-postprocess-split-merge branch 10 times, most recently from 3a84bc9 to dad9483 Compare December 15, 2025 09:56
@thedataking thedataking force-pushed the perl/c2rust-postprocess-split-merge-part-two branch 2 times, most recently from eea8f61 to ec615cf Compare December 15, 2025 12:05
@thedataking thedataking force-pushed the perl/c2rust-postprocess-split-merge branch 2 times, most recently from d3e1971 to 90066a3 Compare December 15, 2025 19:58
@thedataking thedataking changed the base branch from perl/c2rust-postprocess-split-merge to master December 15, 2025 20:40
@thedataking thedataking changed the title postprocess: Support multiple transforms and more postprocess: Support multiple transforms and validation Dec 15, 2025
A later change adds an abstract base class using the `Transform` suffix
so it is more consistent with naming elsewhere in this project to call
the inaugural transform `CommentsTransform`. Updated cache dir
correspondingly.
Adds base class for transforms.
@thedataking thedataking force-pushed the perl/c2rust-postprocess-split-merge-part-two branch from ec615cf to f144132 Compare December 15, 2025 20:44
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