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

Multiple format changes at once #2

Merged
merged 1 commit into from Sep 11, 2022
Merged

Multiple format changes at once #2

merged 1 commit into from Sep 11, 2022

Conversation

Quantum-Codes
Copy link
Contributor

@Quantum-Codes Quantum-Codes commented Sep 10, 2022

Added a for loop and also made it such that you can input both a list and a string into the log_type param to change multiple formats at once.

Also you can edit to change some stuff

not tested

use case:

To change 2 formats:

logger.format_log(log_type='DEBUG', format_string=debug_format_string)
logger.format_log(log_type='ERROR', format_string=debug_format_string)

Now:

logger.format_log(log_type=['DEBUG', 'ERROR'], format_string=debug_format_string)

changing 1 at a time should work as it used to because of line 164

Added a for loop and also made it such that you can input both a list and a string into the `log_type` param to change multiple formats at once.
@Sid72020123
Copy link
Owner

Great Idea! I will soon merge this after testing...

Copy link
Owner

@Sid72020123 Sid72020123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just tested the code. It was a good idea but I had to fix a very small bug...
Merging soon...

@Sid72020123 Sid72020123 merged commit ff5549e into Sid72020123:main Sep 11, 2022
@Quantum-Codes Quantum-Codes deleted the patch-1 branch September 12, 2022 13:09
@Quantum-Codes Quantum-Codes restored the patch-1 branch September 12, 2022 13:09
@Quantum-Codes Quantum-Codes deleted the patch-1 branch September 12, 2022 16:05
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.

None yet

2 participants