Skip to content

Commit

Permalink
yaml files are always indexable
Browse files Browse the repository at this point in the history
Summary:
D66928531 was wrong! The intent was to make sure yaml files are always considered indexable (even though they don't contain defined entities) but it broke the IndexablePredicate.  This caused issue as described in D67335696.

Proposed fixed is to rely on suffix.

Reviewed By: iamirzhan

Differential Revision: D67336679

fbshipit-source-id: 148e6581303bb01b94f8a23a4a76f047f7f5dd3e
  • Loading branch information
Philippe Bidinger authored and facebook-github-bot committed Dec 17, 2024
1 parent 5832426 commit 7acb833
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
2 changes: 2 additions & 0 deletions glean/schema/source/codemarkup.angle
Original file line number Diff line number Diff line change
Expand Up @@ -905,6 +905,8 @@ predicate IndexedFile: src.File
File where
codemarkup.cxx.CxxDataAvailable File |
codemarkup.EntityDataAvailable File |
# All Yaml files are indexable, even though they don't
# contain entities
codemarkup.yaml.YamlDataAvailable File

# Identifies files with indexed entities.
Expand Down
4 changes: 2 additions & 2 deletions glean/schema/source/codemarkup.yaml.angle
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,6 @@ schema codemarkup.yaml.1 {
yaml.XRefViaName { source = S , decl = { just = D } } = XRefs[..];

predicate YamlDataAvailable : src.File
F where
src.File _ = F
F where yaml.XRefsByFile {file = F}

}

0 comments on commit 7acb833

Please sign in to comment.