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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃敡 MAINTAIN: Add a profiler tox env #197

Merged
merged 1 commit into from
Feb 6, 2022

Conversation

hukkin
Copy link
Member

@hukkin hukkin commented Feb 5, 2022

Now that mistletoe beat us (congrats to them!) I was curious to see if there's any easy ways to improve our performance and regain the number one spot 馃槃 I came up with a profiler tox env and thought you may want to merge it to the repository?

@codecov
Copy link

codecov bot commented Feb 5, 2022

Codecov Report

Merging #197 (2001ad3) into master (d6adf66) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #197   +/-   ##
=======================================
  Coverage   96.17%   96.17%           
=======================================
  Files          61       61           
  Lines        3267     3267           
=======================================
  Hits         3142     3142           
  Misses        125      125           
Flag Coverage 螖
pytests 96.17% <酶> (酶)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
螖 = absolute <relative> (impact), 酶 = not affected, ? = missing data
Powered by Codecov. Last update d6adf66...2001ad3. Read the comment docs.

@hukkin hukkin mentioned this pull request Feb 5, 2022
@chrisjsewell chrisjsewell merged commit afe075c into executablebooks:master Feb 6, 2022
@hukkin hukkin deleted the profiler branch February 6, 2022 08:00
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.

None yet

2 participants