Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix linting and TypeScript issues, update test utils and instructions-route test #5764

Open
wants to merge 14 commits into
base: main
Choose a base branch
from

Conversation

neubig
Copy link
Contributor

@neubig neubig commented Dec 23, 2024

This PR fixes linting and TypeScript issues, updates test utils, and improves the instructions-route test. It includes the following changes:

  • Updated test utils to include AuthProvider, UserPrefsProvider, and QueryClientProvider
  • Fixed TypeScript errors in the instructions-route test
  • Replaced jest.mock with vi.mock in the instructions-route test
  • General code cleanup and formatting

To run this PR locally, use the following command:

docker run -it --rm   -p 3000:3000   -v /var/run/docker.sock:/var/run/docker.sock   --add-host host.docker.internal:host-gateway   -e SANDBOX_RUNTIME_CONTAINER_IMAGE=docker.all-hands.dev/all-hands-ai/runtime:c9fe12e-nikolaik   --name openhands-app-c9fe12e   docker.all-hands.dev/all-hands-ai/openhands:c9fe12e

devin-ai-integration bot and others added 14 commits December 14, 2024 16:26
- Add InstructionsPanel with tutorial link support
- Add CreateInstructionsModal for PR creation
- Add MicroagentsPanel with temporary and permanent buttons
- Update ProjectMenuCard to integrate new components
- Add i18n strings for new components

Co-Authored-By: [email protected] <[email protected]>
- Add menu items for viewing and adding instructions
- Add menu items for viewing and adding microagents
- Add i18n strings for new menu items
- Update ProjectMenuCard with new handlers

Co-Authored-By: [email protected] <[email protected]>
- Add TypeScript interfaces for instructions and microagents
- Add API methods for managing repository instructions
- Add API methods for managing microagents
- Follow existing patterns for API integration

Co-Authored-By: [email protected] <[email protected]>
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