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.

Sign up for GitHub

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

Jump to bottom

trunner: add support for pytest #345

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

trunner: add support for pytest #345

wants to merge 2 commits into from

Conversation

mateusz-bloch
Copy link
Member

Description

Motivation and Context

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

How Has This Been Tested?

  • Already covered by automatic testing.
  • New test added: (add PR link here).
  • Tested by hand on: (list targets here).

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing linter checks and tests passed.
  • My changes generate no new compilation warnings for any of the targets.

Special treatment

  • This PR needs additional PRs to work (list the PRs, preferably in merge-order).
  • I will merge this PR by myself when appropriate.

@mateusz-bloch mateusz-bloch force-pushed the mateusz-bloch/add_pytest_support branch 2 times, most recently from 290d271 to 8056f20 Compare April 3, 2024 12:25
@github-actions GitHub Actions
Copy link

github-actions bot commented Apr 3, 2024 •
edited
Loading

Unit Test Results

7 786 tests  +63   7 053 ✅ +45   37m 27s ⏱️ -17s
  452 suites + 9     733 💤 +18 
    1 files   ± 0       0 ❌ ± 0 

Results for commit ec2896d. ± Comparison against base commit a20b474.

♻️ This comment has been updated with latest results.

@mateusz-bloch mateusz-bloch force-pushed the mateusz-bloch/add_pytest_support branch 14 times, most recently from ea04263 to afb9fd9 Compare April 4, 2024 08:23
@mateusz-bloch mateusz-bloch marked this pull request as ready for review April 4, 2024 08:50
@mateusz-bloch mateusz-bloch changed the title trunner: add support for pytests trunner: add support for pytest Apr 4, 2024
@mateusz-bloch mateusz-bloch force-pushed the mateusz-bloch/add_pytest_support branch 2 times, most recently from c713fe0 to 5260942 Compare April 10, 2024 15:16
@mateusz-bloch mateusz-bloch marked this pull request as draft April 12, 2024 13:35
@mateusz-bloch mateusz-bloch force-pushed the mateusz-bloch/add_pytest_support branch 4 times, most recently from 8e740df to 8207461 Compare August 13, 2024 12:18
@mateusz-bloch mateusz-bloch force-pushed the mateusz-bloch/add_pytest_support branch from 8207461 to 5a99d31 Compare August 13, 2024 12:29
@mateusz-bloch mateusz-bloch marked this pull request as ready for review August 13, 2024 12:30
@mateusz-bloch mateusz-bloch force-pushed the mateusz-bloch/add_pytest_support branch from 5a99d31 to 4781107 Compare August 13, 2024 12:32
@mateusz-bloch mateusz-bloch marked this pull request as draft August 19, 2024 13:20
@mateusz-bloch mateusz-bloch force-pushed the mateusz-bloch/add_pytest_support branch 5 times, most recently from 800475d to 4ef0cdd Compare September 27, 2024 15:16
@mateusz-bloch mateusz-bloch force-pushed the mateusz-bloch/add_pytest_support branch from 4ef0cdd to ec2896d Compare September 27, 2024 15:38
@mateusz-bloch mateusz-bloch marked this pull request as ready for review September 27, 2024 15:43
@mateusz-bloch mateusz-bloch force-pushed the mateusz-bloch/add_pytest_support branch 2 times, most recently from 6e5ad12 to ec2896d Compare September 27, 2024 15:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@damianloew damianloew Awaiting requested review from damianloew

@maska989 maska989 Awaiting requested review from maska989

@adamdebek adamdebek Awaiting requested review from adamdebek

@KArkadiusz KArkadiusz Awaiting requested review from KArkadiusz

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@mateusz-bloch