Fix formatting

This commit is contained in:
Saoud Rizwan 2024-12-24 19:41:15 -08:00
parent f5e755eae4
commit 2f9c28ec99
3 changed files with 43 additions and 32 deletions

View file

@ -1,23 +1,28 @@
### Description
<!-- Describe your changes in detail. What problem does this PR solve? -->
### Type of Change
<!-- Put an 'x' in all boxes that apply -->
- [ ] 🐛 Bug fix (non-breaking change which fixes an issue)
- [ ] ✨ New feature (non-breaking change which adds functionality)
- [ ] 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
- [ ] 📚 Documentation update
- [ ] 🐛 Bug fix (non-breaking change which fixes an issue)
- [ ] ✨ New feature (non-breaking change which adds functionality)
- [ ] 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
- [ ] 📚 Documentation update
### Pre-flight Checklist
<!-- Put an 'x' in all boxes that apply -->
- [ ] Changes are limited to a single feature, bugfix or chore (split larger changes into separate PRs)
- [ ] Tests are passing (`npm test`) and code is formatted and linted (`npm run format && npm run lint`)
- [ ] I have reviewed [contributor guidelines](https://github.com/cline/cline/blob/main/CONTRIBUTING.md)
- [ ] Changes are limited to a single feature, bugfix or chore (split larger changes into separate PRs)
- [ ] Tests are passing (`npm test`) and code is formatted and linted (`npm run format && npm run lint`)
- [ ] I have reviewed [contributor guidelines](https://github.com/cline/cline/blob/main/CONTRIBUTING.md)
### Screenshots
<!-- For UI changes, add screenshots here -->
### Additional Notes
<!-- Add any additional notes for reviewers -->

View file

@ -25,42 +25,48 @@ If you're planning to work on a bigger feature, please create an issue first so
Anyone can contribute code to Cline, but we ask that you follow these guidelines to ensure your contributions can be smoothly integrated:
1. **Keep Pull Requests Focused**
- Limit PRs to a single feature or bug fix
- Split larger changes into smaller, related PRs
- Try and limit the PR to a single commit
- Limit PRs to a single feature or bug fix
- Split larger changes into smaller, related PRs
- Try and limit the PR to a single commit
2. **Code Quality**
- Run `npm run lint` to ensure code follows our style guidelines
- Run `npm run format` to format your code with Prettier
- Address any ESLint warnings or errors before submitting
- Follow TypeScript best practices and maintain type safety
- Run `npm run lint` to ensure code follows our style guidelines
- Run `npm run format` to format your code with Prettier
- Address any ESLint warnings or errors before submitting
- Follow TypeScript best practices and maintain type safety
3. **Testing**
- Add tests for new features
- Run `npm test` to ensure all tests pass
- Update existing tests if your changes affect them
- Include both unit tests and integration tests where appropriate
- Add tests for new features
- Run `npm test` to ensure all tests pass
- Update existing tests if your changes affect them
- Include both unit tests and integration tests where appropriate
4. **Documentation**
- Update documentation for any changed functionality
- Update the README if your changes affect usage or setup
- Update documentation for any changed functionality
- Update the README if your changes affect usage or setup
5. **Commit Guidelines**
- Write clear, descriptive commit messages
- Use conventional commit format (e.g., "feat:", "fix:", "docs:")
- Reference relevant issues in commits using #issue-number
- Write clear, descriptive commit messages
- Use conventional commit format (e.g., "feat:", "fix:", "docs:")
- Reference relevant issues in commits using #issue-number
6. **Before Submitting**
- Rebase your branch on the latest main
- Ensure your branch builds successfully
- Double-check all tests are passing
- Review your changes for any debugging code or console logs
- Rebase your branch on the latest main
- Ensure your branch builds successfully
- Double-check all tests are passing
- Review your changes for any debugging code or console logs
7. **Pull Request Description**
- Clearly describe what your changes do
- Include steps to test the changes
- List any breaking changes
- Add screenshots for UI changes
- Clearly describe what your changes do
- Include steps to test the changes
- List any breaking changes
- Add screenshots for UI changes
## Contribution Agreement

View file

@ -129,7 +129,7 @@ const ContextMenu: React.FC<ContextMenuProps> = ({
style={{
padding: "8px 12px",
cursor: isOptionSelectable(option) ? "pointer" : "default",
color:
color:
index === selectedIndex && isOptionSelectable(option)
? "var(--vscode-quickInputList-focusForeground)"
: "",