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

Dn/exo logging #401

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

Dn/exo logging #401

wants to merge 4 commits into from

Conversation

dtnewman
Copy link
Contributor

This PR adds basic logging functionality so that when errors occur, users have the option to send logs to exo to help the team diagnose the issue/s.

Major changes are as follows:

  • Adds the CLI args/options to the readme
  • Adds a script for updating/generating that section of the readme
  • Adds persistent and session-based configuration
  • Logs data in a file within the app_data directory (automatically handles this for mac/linux/windows)... config is also stored there in a json file
  • When the user has an error that crashes the program, it gives them the option to report the error, in which case the logs (which are erased at the start of every session) are sent to Exo's servers

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