Skip to content

Remove the unused OpamSwitchState.files function#6662

Merged
kit-ty-kate merged 1 commit intoocaml:masterfrom
kit-ty-kate:rm-switch-state-files
Aug 29, 2025
Merged

Remove the unused OpamSwitchState.files function#6662
kit-ty-kate merged 1 commit intoocaml:masterfrom
kit-ty-kate:rm-switch-state-files

Conversation

@kit-ty-kate
Copy link
Copy Markdown
Member

Extracted from #6625

Copy link
Copy Markdown
Collaborator

@rjbou rjbou left a comment

Choose a reason for hiding this comment

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

Is there a reason for the remove other than unused ?

@kit-ty-kate
Copy link
Copy Markdown
Member Author

Is there a reason for the remove other than unused ?

yes, #6625 will make it more difficult to implement this function so there is no value to reimplement it if it's unused

@kit-ty-kate kit-ty-kate merged commit 01a62c7 into ocaml:master Aug 29, 2025
46 checks passed
@kit-ty-kate kit-ty-kate deleted the rm-switch-state-files branch August 29, 2025 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants