-
Notifications
You must be signed in to change notification settings - Fork 155
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
Better logging #451
Comments
Anything related to this #413? Probably not all, but it would be nice to update this issue with the missing stuff once that PR is merged. |
I completely agree with this request. Even with the latest version, we experience a total absence of info logs after the repository name validation. Enabling debug logs significantly increases the runtime without making the logs easier to follow. For example, I encountered this issue with the debug log:
The repository name should be included here, and this message could potentially be at the info level, indicating either "Skipping changes" or "Applying requested Autolinks." Another example in the same function we receive a similar "is [object Object]" instead of the repo name: safe-settings/lib/plugins/diffable.js Line 93 in 1fff61c
Additionally, there is insufficient indication of the sync progress during a full sync, I would like to get a sense of the percentage of the progress if possible. |
Prerequisites:
Not related to the UI or API
New Feature
The logging in the success case is pretty poor. It will generate a lot with information about the webhook request but none of the data in the body is included int he log. As a consequence I cannot see which repo it is operating against or which event is triggering the event and I have no idea what settings its resolving and atteampting to apply which makes debugging things difficult when they seem to not work.
The text was updated successfully, but these errors were encountered: