Skip to content

Feature: Logger#14

Merged
avisionh merged 4 commits intomainfrom
feature/logger
Jun 11, 2021
Merged

Feature: Logger#14
avisionh merged 4 commits intomainfrom
feature/logger

Conversation

@avisionh
Copy link
Owner

Summary

This branch adds logging in place of print() statements when running the module.

Changes

The high-level changes made in this PR are:

  1. Replace print() statements with logging.

Please see the commit history for more detailed changes.


Check

  • The test suite passes via running the below:
python -m pytest tests/
  • The GitHub Actions checks pass.

avisionh added 3 commits June 10, 2021 13:58
This is to replace the print statements so we can capture more detail of the module running process.
This is so module run processing can be stored for later viewing.
This is so they can be entered correctly.
@codecov
Copy link

codecov bot commented Jun 10, 2021

Codecov Report

Merging #14 (e443b6b) into main (4d94f4c) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main      #14   +/-   ##
=======================================
  Coverage   76.38%   76.38%           
=======================================
  Files           3        3           
  Lines         144      144           
  Branches       32       32           
=======================================
  Hits          110      110           
  Misses         24       24           
  Partials       10       10           

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 4d94f4c...e443b6b. Read the comment docs.

This is so we can checkthe progress of this module.
@avisionh avisionh merged commit 4f433ed into main Jun 11, 2021
@avisionh avisionh deleted the feature/logger branch June 11, 2021 07:42
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.

1 participant