Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix main #7

Merged
merged 7 commits into from
Apr 17, 2023
Merged

fix main #7

merged 7 commits into from
Apr 17, 2023

Conversation

eleanorjboyd
Copy link
Owner

This logic now successfully works to discover the pytest repo tests.
This branch takes into account all the previous comments made to the
python discovery logic in the previous PR (located on my personal fork).
Therefore this is a second round of edits on this code. It now works for
the pytest library (discovers all the tests in the pytest library).


Co-authored-by: Brett Cannon brcan@microsoft.com
Co-authored-by: Karthik Nadig kanadig@microsoft.com

karthiknadig and others added 7 commits April 11, 2023 12:34
no change logs because this feature is not released yet.

Fixes microsoft#21039
Closes microsoft#20906

---------

Co-authored-by: Courtney Webster <60238438+cwebster-99@users.noreply.github.com>
closes microsoft#20078 and
closes microsoft#20085 (which is
about the testing work to support this code)

This logic now successfully works to discover the pytest repo tests.
This branch takes into account all the previous comments made to the
python discovery logic in the previous PR (located on my personal fork).
Therefore this is a second round of edits on this code. It now works for
the pytest library (discovers all the tests in the pytest library).

---------

Co-authored-by: Brett Cannon <brcan@microsoft.com>
Co-authored-by: Karthik Nadig <kanadig@microsoft.com>
@eleanorjboyd eleanorjboyd merged commit 69c100a into main Apr 17, 2023
@eleanorjboyd eleanorjboyd deleted the fix-main branch April 17, 2023 18:18
eleanorjboyd added a commit that referenced this pull request Apr 18, 2023
eleanorjboyd added a commit that referenced this pull request Apr 18, 2023
eleanorjboyd added a commit that referenced this pull request Apr 18, 2023
eleanorjboyd added a commit that referenced this pull request Apr 18, 2023
eleanorjboyd added a commit that referenced this pull request Apr 18, 2023
eleanorjboyd added a commit that referenced this pull request Apr 19, 2023
eleanorjboyd added a commit that referenced this pull request Apr 19, 2023
eleanorjboyd added a commit that referenced this pull request Apr 19, 2023
eleanorjboyd added a commit that referenced this pull request Apr 19, 2023
eleanorjboyd added a commit that referenced this pull request Apr 19, 2023
eleanorjboyd added a commit that referenced this pull request Apr 19, 2023
eleanorjboyd added a commit that referenced this pull request Apr 19, 2023
eleanorjboyd added a commit that referenced this pull request Apr 19, 2023
eleanorjboyd added a commit that referenced this pull request Apr 19, 2023
eleanorjboyd added a commit that referenced this pull request Apr 19, 2023
eleanorjboyd added a commit that referenced this pull request Apr 19, 2023
eleanorjboyd added a commit that referenced this pull request Apr 19, 2023
eleanorjboyd added a commit that referenced this pull request Apr 19, 2023
eleanorjboyd added a commit that referenced this pull request Apr 19, 2023
eleanorjboyd added a commit that referenced this pull request Apr 19, 2023
eleanorjboyd added a commit that referenced this pull request Apr 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants