Skip to content

fix patternIndices typo #250085

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

Merged
merged 7 commits into from
Jun 6, 2025
Merged

fix patternIndices typo #250085

merged 7 commits into from
Jun 6, 2025

Conversation

matthew-godin
Copy link
Contributor

Rename patternIndecies to patternIndices (typo) in src/workbench/services/search/common/getFileResults.ts

Copy link

@DhruvTheDev1 DhruvTheDev1 left a comment

Choose a reason for hiding this comment

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

Good find!

  • Corrects code typo: Improves code clarity.
  • Change is safe and local: updates one file and one variable declaration
  • All of the typos updated with correct spelling
  • No functional changes

Copy link
Contributor

@osortega osortega left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution!

@vs-code-engineering vs-code-engineering bot added this to the May 2025 milestone Jun 2, 2025
@osortega osortega enabled auto-merge June 5, 2025 17:28
@osortega osortega closed this Jun 5, 2025
auto-merge was automatically disabled June 5, 2025 18:30

Pull request was closed

@osortega osortega reopened this Jun 5, 2025
@osortega osortega closed this Jun 5, 2025
@osortega osortega reopened this Jun 5, 2025
@osortega osortega closed this Jun 6, 2025
@osortega osortega reopened this Jun 6, 2025
@osortega osortega merged commit a5af033 into microsoft:main Jun 6, 2025
11 checks passed
@osortega osortega modified the milestones: May 2025, June 2025 Jun 6, 2025
guilhermescarreira pushed a commit to guilhermescarreira/vscode that referenced this pull request Jun 8, 2025
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.

4 participants