Skip to content

Conversation

kaizencc
Copy link
Contributor

@kaizencc kaizencc commented Jul 14, 2025

We currently send a message per unpublished version. However, a package that has many versions unpublished will pollute the logs with many messages. This PR consolidates those messages into one message that can be collapsed so that the logs are much easier to parse.

No tests; this only affects logging


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

}
}

for (const version of unpublishedVersions) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will still result in a single log per version. You'll need to concat the messages into a single INFO log to reduce the number of logs.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:(

Copy link
Contributor

@iankhou iankhou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Solution doesn't reduce number of logs, see my comment

@kaizencc kaizencc added this pull request to the merge queue Jul 15, 2025
Merged via the queue into main with commit e7fe2b7 Jul 15, 2025
6 checks passed
@kaizencc kaizencc deleted the conroy/logs branch July 15, 2025 14:50
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.

2 participants