Skip to content

Conversation

alex-courtis
Copy link
Member

@alex-courtis alex-courtis commented Sep 8, 2025

fixes #3196

  • remove utils.read_file
  • move utils.move_missing_val and table_create_missing to legacy as it is the only consumer

No other obvious removals / refactors stood out.

@alex-courtis
Copy link
Member Author

Many thanks for bringing this to our attention @sepruko

@alex-courtis alex-courtis merged commit f92cc3a into master Sep 8, 2025
6 checks passed
@alex-courtis alex-courtis deleted the 3196-remove-utils-read-file branch September 8, 2025 01:12
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.

utils.read_file may hold onto fd
1 participant