Loading...
Loading...
Open source contribution best practices. Creating quality pull requests, writing good issues, following project conventions, and collaborating effectively with maintainers.
npx skill4agent add terraphim/terraphim-skills open-source-contribution[ ] Read CONTRIBUTING.md if it exists
[ ] Review CODE_OF_CONDUCT.md
[ ] Check existing issues for duplicates
[ ] Understand the project's goals and scope
[ ] Review recent PRs for conventions
[ ] Check if the feature/fix is wanted# Fork the repository
gh repo fork owner/project --clone
# Set up upstream remote
git remote add upstream https://github.com/owner/project.git
# Install dependencies and verify tests pass
cargo build
cargo test## Description
[Clear, concise description of the bug]
## Steps to Reproduce
1. [First step]
2. [Second step]
3. [Third step]
## Expected Behavior
[What should happen]
## Actual Behavior
[What actually happens]
## Environment
- OS: [e.g., Ubuntu 22.04]
- Rust version: [e.g., 1.75.0]
- Project version: [e.g., v1.2.3 or commit hash]
## Additional Context
[Screenshots, logs, related issues]
## Possible Solution (optional)
[If you have ideas about what might fix this]## Problem Statement
[What problem does this solve? Who benefits?]
## Proposed Solution
[How would this feature work?]
## Alternatives Considered
[What other solutions did you consider?]
## Additional Context
[Mockups, examples from other projects, etc.]
## Willingness to Contribute
[ ] I am willing to implement this feature
[ ] I need help implementing this feature
[ ] I am only suggesting this featurefeature/add-caching-layer
fix/handle-empty-input
docs/update-readme
refactor/simplify-parserfeat: add support for custom delimiters
Add the ability to specify custom delimiters when parsing CSV files.
This is useful for tab-separated values and other formats.
Closes #123type: subjectfeatfixdocsstylerefactortestchore## Summary
[Brief description of changes]
## Motivation
[Why is this change needed?]
## Changes
- [Change 1]
- [Change 2]
- [Change 3]
## Testing
[How were these changes tested?]
## Checklist
- [ ] Tests pass locally
- [ ] Code follows project style
- [ ] Documentation updated
- [ ] CHANGELOG updated (if required)
- [ ] Commit messages follow conventions
## Related Issues
Closes #[issue number]| Size | Lines Changed | Review Time |
|---|---|---|
| XS | < 10 | Minutes |
| S | 10-100 | < 1 hour |
| M | 100-500 | Hours |
| L | 500-1000 | Days |
| XL | > 1000 | Split recommended |
Before Submitting:
[ ] cargo fmt has been run
[ ] cargo clippy shows no warnings
[ ] cargo test passes
[ ] New code has tests
[ ] Documentation is updated
[ ] No unrelated changes included
[ ] Commit history is clean# Acknowledging feedback
> Consider using `match` instead of `if let` here
Good point! Updated in abc1234.
# Asking for clarification
> This could be simplified
Could you elaborate? I'm not sure if you mean [option A] or [option B].
# Respectfully disagreeing
> Remove this error check
I'd prefer to keep this because [reason]. The error can occur when [scenario]. Happy to discuss further if you disagree.# Sync with upstream
git fetch upstream
git checkout main
git merge upstream/main
git push origin main
# Rebase feature branch
git checkout feature/my-feature
git rebase main