Skip to content

Conversation

@mildwonkey
Copy link
Contributor

I expect these will come back, but since we don't have a firm roadmap just yet it's better not to make future promises.
This removes the destroy_ events from the list of supported events and drops the diagnostic referencing their future implementation.

Fixes #

Target Release

1.14.x

Rollback Plan

  • If a change needs to be reverted, we will roll out an update to the code within 7 days.

Changes to Security Controls

Are there any changes to security controls (access controls, encryption, logging) in this pull request? If so, explain.

CHANGELOG entry

  • This change is user-facing and I added a changelog entry.
  • This change is not user-facing.

@mildwonkey mildwonkey requested a review from a team as a code owner September 10, 2025 14:14
@mildwonkey mildwonkey force-pushed the mildwonkey/destroy-events-never-heard-of-em branch from e5976bd to 329762b Compare September 10, 2025 14:15
@mildwonkey mildwonkey added the no-changelog-needed Add this to your PR if the change does not require a changelog entry label Sep 10, 2025
I expect these will come back, but since we don't have a firm roadmap just yet it's better not to make future promises.
@mildwonkey mildwonkey force-pushed the mildwonkey/destroy-events-never-heard-of-em branch from 329762b to c11df4c Compare September 10, 2025 14:22
@mildwonkey mildwonkey merged commit e315a07 into main Sep 10, 2025
7 checks passed
@mildwonkey mildwonkey deleted the mildwonkey/destroy-events-never-heard-of-em branch September 10, 2025 14:47
@github-actions
Copy link
Contributor

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 11, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

no-changelog-needed Add this to your PR if the change does not require a changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants