Skip to content

Replaced print statements with logger #1299

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Open
wants to merge 3 commits into
base: dev
Choose a base branch
from
Open

Replaced print statements with logger #1299

wants to merge 3 commits into from

Conversation

dmytrotsko
Copy link
Contributor

Summary:

Replaced print statements with logger

Prerequisites:

  • Branch is up-to-date with the branch to be merged with, i.e. dev
  • Build is successful
  • Code is cleaned up and formatted

@dmytrotsko dmytrotsko requested a review from melange396 October 6, 2023 08:53
@dmytrotsko dmytrotsko linked an issue Oct 6, 2023 that may be closed by this pull request
Use logger in src/acquisition/* code #1294
Open
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

No Coverage information No Coverage information
2.3% 2.3% Duplication

@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

1 New issue
0 Security Hotspots
No data about Coverage
2.3% Duplication on New Code

See analysis details on SonarCloud

# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@melange396 melange396 Awaiting requested review from melange396

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Use logger in src/acquisition/* code
1 participant
@dmytrotsko