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

Allow adjustment of logging level #133

Merged
merged 2 commits into from
Apr 26, 2023
Merged

Conversation

sverrirarnors
Copy link
Contributor

Allow logging level to be passed to the constructor, see #132

@akaszynski akaszynski closed this Apr 26, 2023
@akaszynski akaszynski reopened this Apr 26, 2023
@akaszynski
Copy link
Owner

Closed/reopened to enable CI to run (hopefully) on external branch.

@akaszynski
Copy link
Owner

Local testing passes:

==================================== test session starts =====================================
platform linux -- Python 3.11.3, pytest-7.2.2, pluggy-1.0.0
PySide6 6.4.2 -- Qt runtime 6.4.2 -- Qt compiled 6.4.2
rootdir: /home/alex/source/keepa, configfile: pytest.ini
plugins: reverse-1.5.0, anyio-3.6.2, cov-4.0.0, pytest_pyvista-0.1.8, xdist-3.1.0, repeat-0.9.1, qt-4.2.0, hypothesis-6.61.0, memprof-0.2.0, sphinx-0.5.0, asyncio-0.20.3
asyncio: mode=Mode.STRICT
collected 54 items                                                                           

tests/test_async_interface.py .........................                                [ 46%]
tests/test_interface.py .............................                                  [100%]

---------- coverage: platform linux, python 3.11.3-final-0 -----------
Coverage HTML written to dir htmlcov

Required test coverage of 90% reached. Total coverage: 90.18%
========================= 54 passed, 3 warnings in 68.12s (0:01:08) ==========================

Merging. Thanks for your contribution!

@akaszynski akaszynski merged commit 2a33477 into akaszynski:master Apr 26, 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