-
Notifications
You must be signed in to change notification settings - Fork 10.8k
fix(patch): cherry-pick addb57c to release/v0.26.0-preview.2-pr-17331 [CONFLICTS] #17345
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
Conversation
Summary of ChangesHello @gemini-cli-robot, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request attempts to backport a critical fix related to keypress handling in the CLI, specifically ensuring that a 'return' keypress is correctly interpreted as a newline rather than a submission. Due to merge conflicts encountered during the automated cherry-pick process, manual intervention is required to resolve these conflicts before the fix can be integrated into the Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code Review
This pull request appears to be an attempt to resolve merge conflicts from a cherry-pick, but it contains unresolved merge conflict markers in packages/cli/src/ui/contexts/KeypressContext.test.tsx. These markers will break the build and must be removed. Additionally, both the implementation in KeypressContext.tsx and the corresponding test are attempting to use alt and cmd properties on the Key type, which are not defined. The alt modifier should be handled via the meta property, and cmd is not a supported modifier. I've provided critical feedback with suggestions to fix these issues.
| <<<<<<< HEAD | ||
| ======= | ||
| shift: true, | ||
| alt: false, | ||
| ctrl: false, | ||
| cmd: false, | ||
| >>>>>>> addb57c31 (Emit correct newline type return (#17331)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file contains unresolved merge conflict markers which will cause syntax errors. Additionally, the properties alt and cmd are not part of the Key type and will cause the test to fail. The alt modifier should be checked via the meta property, and cmd is not a valid property on the Key type.
shift: true,
ctrl: false,
meta: false,
| alt: false, | ||
| ctrl: false, | ||
| cmd: false, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The Key type does not have alt or cmd properties. The alt modifier is represented by the meta property, and the cmd property is not supported on the Key type. Using these properties will cause a TypeScript compilation error.
| alt: false, | |
| ctrl: false, | |
| cmd: false, | |
| meta: false, | |
| ctrl: false, |
02f11fc to
296ca12
Compare
|
Size Change: +111 B (0%) Total Size: 23.2 MB ℹ️ View Unchanged
|
ee87c98
into
release/v0.26.0-preview.2-pr-17331
This PR automatically cherry-picks commit addb57c to patch version v0.26.0-preview.2 in the preview release to create version 0.26.0-preview.3.
This cherry-pick resulted in merge conflicts that need manual resolution.
🔧 Next Steps:
📋 Files with conflicts:
The commit has been created with conflict markers for easier manual resolution.
🚨 Important: