New issue
Advanced search Search tips

Issue 888054 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Sep 26
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Issue report for SpellChecker (Tast, not taste)

Project Member Reported by bpastene@chromium.org, Sep 21

Issue description

https://tricium-prod.appspot.com/run/5718595359211520

Can we add the word "tast" to an exclusion in the spell checker? It's a type of chromeos test that we're starting to enable in chromium/src, and tricium falsely flags it as a typo:
https://chromium-review.googlesource.com/c/chromium/src/+/1237673/3/build/chromeos/run_vm_test.py#526

 
Cc: -diegomtzg@google.com mar...@chromium.org
Status: Available (was: Untriaged)
Cc: -mar...@chromium.org qyears...@chromium.org
Owner: mar...@chromium.org
Status: Assigned (was: Available)
Will do.
Project Member

Comment 3 by bugdroid1@chromium.org, Sep 24

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/ce2e177b3447d66c7a1eefb9c0fd6d7864ca82d5

commit ce2e177b3447d66c7a1eefb9c0fd6d7864ca82d5
Author: Marc-Antoine Ruel <maruel@chromium.org>
Date: Mon Sep 24 17:23:32 2018

Add 'tast' to ignore list

R=jchinlee@chromium.org

Bug:  888054 
Change-Id: Icc33c32a838d2a35e2fde9219c124eeda0ba61ed
Reviewed-on: https://chromium-review.googlesource.com/1239020
Reviewed-by: Quinten Yearsley <qyearsley@chromium.org>
Commit-Queue: Marc-Antoine Ruel <maruel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#17821}
[modify] https://crrev.com/ce2e177b3447d66c7a1eefb9c0fd6d7864ca82d5/go/src/infra/tricium/functions/spellchecker/spellchecker.go

Sign in to add a comment