Setting Entity Query trash context to 'ignore' manually should not alter entity queries

Problem/Motivation

If I set the current trash context to 'ignore', the code in \Drupal\trash\Hook\TrashEntityHooks::entityQueryAlter() still alters my query. Should it return as a no-op and not add any conditions when this is the case? I'd like to just manually add $query->addMetadata('trash', 'ignore') and not have to use \Drupal::service('trash.manager')->executeInTrashContext() as a wrapper.

Steps to reproduce

$query = \Drupal::entityTypeManager()->getQuery('node');
$query->addMetadata('trash', 'ignore');
$nids = $query->execute();

Confirm that $nids does not contain any trashed node IDs.

Proposed resolution

Have \Drupal\trash\Hook\TrashEntityHooks::entityQueryAlter() return without modifying the query when $query->getMetadata('trash') === 'ignore' without having to use executeInTrashContext()

Remaining tasks

User interface changes

None

API changes

None

Data model changes

None