Skip to content

Conversation

@bjk7119
Copy link
Contributor

@bjk7119 bjk7119 commented Jun 18, 2021

Description

Delete unnecessary lines at tox.ini

Type of change

Please insert 'x' one of the type of change.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Documentation update
  • Refactoring, Maintenance
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

@bjk7119 bjk7119 added the chore [PR/Issue] Refactoring, maintenance the code label Jun 18, 2021
@bjk7119 bjk7119 requested review from dd-jy and soimkim June 18, 2021 09:38
@soimkim
Copy link
Contributor

soimkim commented Jun 19, 2021

Dear @bjk7119 ,

If you look at the tox result printed on the Checks tab, the following row is displayed.
Isn't this a logging level related bug?

[ DEBUG] Error! This message is a level that should not be displayed on the console. [ INFO] Error! This message is a level that should not be displayed on the console.

@bjk7119
Copy link
Contributor Author

bjk7119 commented Jun 21, 2021

It's not a bug.
It happened by related file logging level. I will modify those logs will not be visible.

@soimkim
Copy link
Contributor

soimkim commented Jun 24, 2021

@bjk7119 ,
Please update it later so that it can be distinguished whether the log that comes out of the test is an error or not.

@bjk7119 bjk7119 merged commit d64a36b into main Jun 24, 2021
@soimkim soimkim deleted the develop branch June 24, 2021 05:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore [PR/Issue] Refactoring, maintenance the code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants