Skip to content
This repository was archived by the owner on Aug 7, 2024. It is now read-only.

Add utility for filtering out skipped tests in large cross-product groups #303

Open
wants to merge 4 commits into
base: gh/drisspg/3/base
Choose a base branch
from

Conversation

drisspg
Copy link
Contributor

@drisspg drisspg commented Jul 3, 2024

Summary

Quick clean up to tests:
Before:

================================================================ 271 passed, 168 skipped, 42 warnings in 6.58s ================================================================

After:

============================================================================= 271 passed in 5.79s =============================================================================

Stack from ghstack (oldest at bottom):

[ghstack-poisoned]
drisspg added a commit that referenced this pull request Jul 3, 2024
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jul 3, 2024
drisspg added a commit that referenced this pull request Jul 3, 2024
[ghstack-poisoned]
drisspg added a commit that referenced this pull request Jul 3, 2024
[ghstack-poisoned]
drisspg added a commit that referenced this pull request Jul 3, 2024
@drisspg drisspg changed the title Add utility for filtering out skpped tests in large paremtrization groups Add utility for filtering out skipped tests in large cross-product groups Jul 3, 2024
…-product groups"

# Summary
Quick clean up to tests:
Before:
```Shell
================================================================ 271 passed, 168 skipped, 42 warnings in 6.58s ================================================================
```
After:
```Shell
============================================================================= 271 passed in 5.79s =============================================================================
```




[ghstack-poisoned]
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants