Skip to content

Comments

🚀 feat(storage/writer) add validate_required_infos before creating and pushing heavy data to prevent later error#311

Merged
casenave merged 3 commits intomainfrom
fix/issue-310
Feb 19, 2026
Merged

🚀 feat(storage/writer) add validate_required_infos before creating and pushing heavy data to prevent later error#311
casenave merged 3 commits intomainfrom
fix/issue-310

Conversation

@casenave
Copy link
Member

@casenave casenave commented Feb 18, 2026

Checklist

  • Typing enforced
  • Documentation updated
  • Changelog updated
  • Tests and Example updates
  • Coverage should be 100%

🔗 Related issues (optional)

Closes #310

@casenave casenave requested a review from a team as a code owner February 18, 2026 19:39
@codecov
Copy link

codecov bot commented Feb 18, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@casenave
Copy link
Member Author

Hi @JeoaFesketto, thank you for having tested the lib with a large dataset and proposing issue #310. Do you think that this PR addresses your issue ?

@JeoaFesketto
Copy link

Yes, I believe it does, thank you!

@casenave casenave merged commit 299702e into main Feb 19, 2026
13 checks passed
@casenave casenave deleted the fix/issue-310 branch February 19, 2026 09:49
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.

Check Infos, metadata and problem definition before push_local_to_hub in push_to_hub

3 participants