Skip to content

Conversation

@leoschwarz
Copy link
Collaborator

@leoschwarz leoschwarz commented Dec 8, 2025

  • changelog
  • testing
  • settle on api (excel in the name)
  • dependency: if excel is not available, omit...

@github-actions
Copy link

github-actions bot commented Dec 8, 2025

📝 "TODO" Changes Detected

Summary: ➕ 5 "TODO"s added

➕ Added "TODO"s (5)

  • bfabric/src/bfabric/experimental/dataset_changes.py:61: # TODO future extensions: - dectect rename (covered by add/remove right now)
  • bfabric/src/bfabric/experimental/dataset_changes.py:90: # TODO in principle, there is also strict=False which is possibly a bit dangerous in this context
  • bfabric/src/bfabric/experimental/dataset_changes.py:92: # TODO find the correct exception class: https://docs.pola.rs/api/python/stable/reference/exceptions.html
  • bfabric_scripts/src/bfabric_scripts/cli/dataset/update.py:50: # TODO we could check that the result of the conversion has correct types in the dict repr
  • bfabric/src/bfabric/experimental/dataset_changes.py:76: # TODO this code assumes identical columns, if generalized it should only operate on the common ones

This comment is automatically updated when "TODO" changes are detected.

@leoschwarz leoschwarz mentioned this pull request Dec 11, 2025
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