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

Improve natlab logs #1149

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Improve natlab logs #1149

wants to merge 6 commits into from

Conversation

lcruz99
Copy link
Contributor

@lcruz99 lcruz99 commented Mar 4, 2025

Problem

With #1143 the nat-lab logs were cleaned, during that task execution it was realized that the logs were still too verbose, many adding trivial information.

Nat-lab could also make use of a logger to standardize logs format and providing the user the option to choose the amount of verbosity.

Solution

Remove trivial logs and add a few others, mainly add a logger.

☑️ Definition of Done checklist

  • Commit history is clean (requirements)
  • README.md is updated
  • Functionality is covered by unit or integration tests

@lcruz99 lcruz99 force-pushed the LLT-6027_cleanup_natlab_logs branch from c754660 to f5e6a0d Compare March 4, 2025 13:27
@lcruz99 lcruz99 changed the base branch from LLT-6027_cleanup_natlab_logs to main March 4, 2025 13:27
@lcruz99 lcruz99 changed the base branch from main to LLT-6027_cleanup_natlab_logs March 4, 2025 13:28
@lcruz99 lcruz99 force-pushed the improve_natlab_logs branch from 1b1e7e6 to 9b042bf Compare March 4, 2025 13:37
@lcruz99 lcruz99 force-pushed the improve_natlab_logs branch from 9b042bf to a1515c6 Compare March 4, 2025 16:14
@lcruz99 lcruz99 force-pushed the improve_natlab_logs branch from a1515c6 to 1f8d28f Compare March 4, 2025 17:07
@lcruz99 lcruz99 force-pushed the improve_natlab_logs branch from 1f8d28f to f70a8d7 Compare March 4, 2025 17:53
@lcruz99 lcruz99 marked this pull request as ready for review March 4, 2025 18:06
@lcruz99 lcruz99 requested a review from a team as a code owner March 4, 2025 18:07
@lcruz99 lcruz99 force-pushed the LLT-6027_cleanup_natlab_logs branch from f5e6a0d to 318c115 Compare March 7, 2025 10:26
Base automatically changed from LLT-6027_cleanup_natlab_logs to main March 7, 2025 13:44
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