deps: Bump updatecli version to v0.89.0 #304
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
![elastic-observability-automation[bot]](https://avatars.githubusercontent.com/u/6764390?s=60&v=4){kind=small}
Made with ❤️️ by updatecli
elastic-observability-automation
bot
added
the
dependencies
v1v
approved these changes
Dec 18, 2024
trentm
added a commit
to elastic/elastic-otel-node
that referenced
this pull request
Dec 19, 2024
I am proposing this as an alternative to the issue and PR Slack notifications we currently get from https://github.com/integrations/slack because of the following issues with that integration: - It is over-wordy, resulting in notification noise masking any other conversation in the channel. This lowers the signal-to-noise ratio in the channel, which discourages meaningful conversation. - It includes some body content from issues/PRs. This may be a nice idea for a repo with commonly simple and short issue/PR descriptions. However, in general it is just spam. It can also get silly for PRs that have large `<details>` blocks, which the Slack rendering doesn't hide. E.g. elastic/opbeans-node#304 - It includes some issue metadata, sometimes some chat-ops buttons ("Comment", "Close"). - It includes a variable-height summary of actions/checks on PRs. This sounds nice, but it results in dynamic-height content in the chat area that discourages ongoing conversation. - It `@-`names people listed as reviewers for PRs and makes a *threads* of every PR to show when it is merged. This sounds like a nice idea, but in practice means that the "Threads" section of Slack is spammed with useless unread threads for merged PRs that I've reviewed. - Basically the Slack integration is trying to be a fancy full-on chat-ops thingy, and I don't want the noise. I want higher signal. - It doesn't support ignoring some PR categories: specifically ignoring dependabot/renovate PRs. Currently we have those scheduled for Sunday. Every Monday morning is a multi-page scroll of useless notifications about regular dependabot maintenance work. It is very easy to lose *meaningful* conversation from Friday or over the weekend in that noise.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
deps: Bump updatecli version
deps(updatecli): Bump updatecli version to v0.89.0
1 file(s) updated with "updatecli v0.89.0": * .tool-versions
v0.89.0
Created automatically by Updatecli
Options:
Most of Updatecli configuration is done via its manifest(s).
Feel free to report any issues at github.com/updatecli/updatecli.
If you find this tool useful, do not hesitate to star our GitHub repository as a sign of appreciation, and/or to tell us directly on our chat!