Skip to content

Add support for returning values#12

Merged
art049 merged 4 commits into
CodSpeedHQ:masterfrom
patrick91:feature/return-value
Jul 19, 2023
Merged

Add support for returning values#12
art049 merged 4 commits into
CodSpeedHQ:masterfrom
patrick91:feature/return-value

Conversation

@patrick91
Copy link
Copy Markdown
Contributor

No description provided.

Comment thread src/pytest_codspeed/plugin.py
@art049 art049 marked this pull request as ready for review July 12, 2023 10:48
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Jul 12, 2023

CodSpeed Performance Report

Merging #12 will not alter performance

Comparing patrick91:feature/return-value (d6c492e) with master (48e4228)

Summary

✅ 5 untouched benchmarks

Comment thread src/pytest_codspeed/plugin.py
Comment thread tests/test_return_value.py Outdated
@patrick91 patrick91 requested a review from art049 July 12, 2023 14:39
@art049
Copy link
Copy Markdown
Member

art049 commented Jul 19, 2023

Awesome, thanks a lot @patrick91 ! This use case is super interesting to transform a test in a benchmark and benefit from the both worlds!

I don't know if it's supported in pytest-speed / pytest-benchmarks but it would be awesome to have it there as well, I might work on something :)

@art049 art049 merged commit ab2d827 into CodSpeedHQ:master Jul 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