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

Support logs to support debugging #6

Open
2 tasks
rash805115 opened this issue May 24, 2024 · 0 comments
Open
2 tasks

Support logs to support debugging #6

rash805115 opened this issue May 24, 2024 · 0 comments
Labels
enhancement New feature or request octo For packages/octo project.

Comments

@rash805115
Copy link
Contributor

###Overview
Currently there are no logs being generated from running Octo. This can be quite confusing for everyone who wants to understand why something didn't run, or even to understand what is happening in the system.

Technical Considerations

  • In some places, console.error() is being used.
  • Most errors being thrown are very generic, without any identifying information.
  • There are no DEBUG/INFO logs.
  • Should customers/users be able to control the amount of logs being generated, by choosing the level of logging?
  • Tests such as action.decorator.spec.ts can use the logs to assert on testing.

Acceptance Criteria

  • Logging library introduced, and appropriate logs are generated on every action Octo does.
  • Tests depending on this feature should be fixed by adding log assertions.
@rash805115 rash805115 added octo For packages/octo project. enhancement New feature or request labels May 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request octo For packages/octo project.
Projects
None yet
Development

No branches or pull requests

1 participant