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

Incorporate `python_db_logger` #134

Open

michael-bentz opened this issue Feb 9, 2023 · 0 comments

Contributor

michael-bentz commented Feb 9, 2023

Add python_db_logger to Nacculator:

Add an example.env with the necessary DB_* environment variables
Add a local mysql container definition. Copy the folder structure (.containers/mysql) and docker-compose.yml from python_db_logger
Update README with instructions on how to start the docker container
Add python_db_logger as a dependency in setup.py:

install_requires=[
      "python_db_logger @ git+ssh://[email protected]:/ctsit/python_db_logger.git"
  ]

Create a configure_logging function similar to https://github.com/ctsit/ufvivo-pubingest/blob/0779cf62b41245c479ebf49a9233d77bd644ab58/ufvivo/pubingest/__main__.py#L89-L111 that also creates an instance of DBLogger
Replace all calls to print with db_logger.print_info() , db_logger.print_warn() , etc.

The text was updated successfully, but these errors were encountered:

rtdtwo mentioned this issue

Add Python DB Logger #136

Merged

8 tasks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment