Skip to content

Conversation

@shaffeeullah
Copy link
Contributor

Tests are not expected to pass until the feature is released.

@shaffeeullah shaffeeullah requested review from a team as code owners April 6, 2022 20:15
@product-auto-label product-auto-label bot added size: m Pull request size is medium. api: storage Issues related to the googleapis/nodejs-storage API. labels Apr 6, 2022
@shaffeeullah shaffeeullah requested a review from ddelgrosso1 April 6, 2022 21:02
Comment on lines 1345 to 1346
(rule.condition.matchesPrefix as string[]).length === 1 &&
(rule.condition.matchesPrefix as string[])[0] === TESTS_PREFIX
Copy link
Contributor

Choose a reason for hiding this comment

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

Optional: You can use Array.isArray here to avoid casting

@ddelgrosso1
Copy link
Contributor

Should this be marked do not merge until the feature is ready?

@shaffeeullah shaffeeullah added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Apr 7, 2022
@shaffeeullah shaffeeullah removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jun 7, 2022
@product-auto-label product-auto-label bot added size: s Pull request size is small. and removed size: m Pull request size is medium. labels Jun 7, 2022
@shaffeeullah shaffeeullah merged commit c22984c into main Jun 8, 2022
@shaffeeullah shaffeeullah deleted the shaffeeullah/olmPrefixSuffix branch June 8, 2022 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: storage Issues related to the googleapis/nodejs-storage API. size: s Pull request size is small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants