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.

Sign up for GitHub

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

Jump to bottom

Include log panics crate, improve cofiguration load and unit tests #100

Merged
merged 6 commits into from
Mar 8, 2023
Merged

Include log panics crate, improve cofiguration load and unit tests #100

merged 6 commits into from
Mar 8, 2023

Conversation

okynos
Copy link
Member

@okynos okynos commented Mar 7, 2023

Hello!

This PR closes #97, #98 and #99

We have added the following:

  • Added log-panics crate to log any error to defined log.
  • Added init method in main to initialize the configuration.
  • Moved configuration to static zone (required to next issues).
  • Moved configuration setup to main init function.
  • Added specific configuration unit tests.

@okynos okynos self-assigned this Mar 7, 2023
@okynos okynos merged commit 8f22f0d into main Mar 8, 2023
@okynos okynos deleted the 97-log-panics branch March 8, 2023 09:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@okynos okynos

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Include log-panics crate to record panics to log file
1 participant
@okynos