Skip to content

Commit

Permalink
TestTargetProperties: fix typo in cpu reservation
Browse files Browse the repository at this point in the history
This is just a comment fix, but as I was looking through the code
to see how this tag is implemented, the difference between this
comment and the [documented syntax](https://docs.bazel.build/versions/master/test-encyclopedia.html#other-resources)
confused me.

Closes bazelbuild#12243.

PiperOrigin-RevId: 337231628
  • Loading branch information
dhalperi authored and copybara-github committed Oct 15, 2020
1 parent 9d29464 commit 0218270
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -165,7 +165,7 @@ public ResourceSet getLocalResourceUsage(Label label, boolean usingLocalTestJobs

ResourceSet testResourcesFromSize = TestTargetProperties.getResourceSetFromSize(size);

// Tests can override their CPU reservation with a "cpus:<n>" tag.
// Tests can override their CPU reservation with a "cpu:<n>" tag.
ResourceSet testResourcesFromTag = null;
for (String tag : executionInfo.keySet()) {
try {
Expand Down

0 comments on commit 0218270

Please sign in to comment.