Skip to content

Fix typo: quote_ahead_fount_at -> quote_ahead_found_at#9280

Merged
captainsafia merged 3 commits intowarpdotdev:masterfrom
filippostanghellini:fix/typo-quote-ahead-variable
Apr 28, 2026
Merged

Fix typo: quote_ahead_fount_at -> quote_ahead_found_at#9280
captainsafia merged 3 commits intowarpdotdev:masterfrom
filippostanghellini:fix/typo-quote-ahead-variable

Conversation

@filippostanghellini
Copy link
Copy Markdown
Contributor

Corrects a typo in the variable name used for tracking quote positions in vim text object operations.

Description

Fixes a typo: quote_ahead_fount_atquote_ahead_found_at in crates/vim/src/text_objects/quote.rs.

Testing

  • No functional changes, purely a variable rename
  • Code compiles successfully

No new tests added as this is a simple variable rename with no behavioral changes.

Server API dependencies

  • N/A

Agent Mode

  • Warp Agent Mode

Changelog Entries for Stable

N/A - Minor typo fix.

Corrects a typo in the variable name used for tracking quote positions
in vim text object operations.
@cla-bot
Copy link
Copy Markdown

cla-bot Bot commented Apr 28, 2026

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have the users @filippostanghellini on file. In order for us to review and merge your code, each contributor must visit https://cla.warp.dev to read and agree to our CLA. Once you have done so, please comment @cla-bot check to trigger another check.

@oz-for-oss
Copy link
Copy Markdown
Contributor

oz-for-oss Bot commented Apr 28, 2026

@filippostanghellini

I'm starting a first review of this pull request.

You can follow along in the session on Warp.

I approved this pull request and requested human review from: @vorporeal, @alokedesai, @zachbai.

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

Copy link
Copy Markdown
Contributor

@oz-for-oss oz-for-oss Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overview

This PR consistently renames the local variable quote_ahead_fount_at to quote_ahead_found_at in crates/vim/src/text_objects/quote.rs without changing the vim quote text-object behavior.

Concerns

  • No correctness, security, error-handling, or performance concerns found in the changed lines.

Verdict

Found: 0 critical, 0 important, 0 suggestions

Approve

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

@cla-bot
Copy link
Copy Markdown

cla-bot Bot commented Apr 28, 2026

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have the users @filippostanghellini on file. In order for us to review and merge your code, each contributor must visit https://cla.warp.dev to read and agree to our CLA. Once you have done so, please comment @cla-bot check to trigger another check.

@filippostanghellini
Copy link
Copy Markdown
Contributor Author

@cla-bot check

@cla-bot
Copy link
Copy Markdown

cla-bot Bot commented Apr 28, 2026

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have the users @filippostanghellini on file. In order for us to review and merge your code, each contributor must visit https://cla.warp.dev to read and agree to our CLA. Once you have done so, please comment @cla-bot check to trigger another check.

@cla-bot
Copy link
Copy Markdown

cla-bot Bot commented Apr 28, 2026

The cla-bot has been summoned, and re-checked this pull request!

@filippostanghellini
Copy link
Copy Markdown
Contributor Author

@cla-bot check

@cla-bot cla-bot Bot added the cla-signed label Apr 28, 2026
@cla-bot
Copy link
Copy Markdown

cla-bot Bot commented Apr 28, 2026

The cla-bot has been summoned, and re-checked this pull request!

@captainsafia captainsafia enabled auto-merge (squash) April 28, 2026 19:23
@captainsafia captainsafia merged commit 84592e2 into warpdotdev:master Apr 28, 2026
22 checks passed
@captainsafia captainsafia added the external-contributor Indicates that a PR has been opened by someone outside the Warp team. label Apr 30, 2026 — with Warp Dev Github Integration
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed external-contributor Indicates that a PR has been opened by someone outside the Warp team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants