docs: add docstrings to create_repo function#682
docs: add docstrings to create_repo function#682singhvishalkr wants to merge 1 commit intoDagsHub:mainfrom
Conversation
📝 WalkthroughWalkthroughUpdated the docstring of the Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Poem
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
dagshub/upload/wrapper.py (1)
116-119: Clarify the exact “empty repo” call pattern.Consider explicitly stating that
template="none"alone is not enough with current defaults (gitignores="Python"), and users must also pass emptygitignores/license/readmeto keep it empty.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 5a08f0b9-08e3-4cfe-8812-885d5c3bbb31
📒 Files selected for processing (1)
dagshub/upload/wrapper.py
📜 Review details
🔇 Additional comments (1)
dagshub/upload/wrapper.py (1)
92-138: Great docstring expansion and behavior alignment.This is a strong documentation upgrade: parameter semantics, template behavior, failure modes, and usage examples are clear and match the implementation.
Description
Adds comprehensive docstrings to the
create_repofunction, documenting parameters, return values, and usage.Closes #222
How to test
Verify the docstring is displayed correctly.