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

runtime: add an example for AddCleanup #72795

Closed
cagedmantis opened this issue Mar 11, 2025 · 4 comments
Closed

runtime: add an example for AddCleanup #72795

cagedmantis opened this issue Mar 11, 2025 · 4 comments
Assignees
Labels
compiler/runtime Issues related to the Go compiler and/or runtime. Documentation Issues describing a change to documentation. NeedsFix The path to resolution is known, but the work has not been done.
Milestone

Comments

@cagedmantis
Copy link
Contributor

cagedmantis commented Mar 11, 2025

The runtime.AddCleanup functionality would benefit from having an example. @streamdp added an example in CL 649995. We should backport that example to go1.24.

@gopherbot gopherbot added the compiler/runtime Issues related to the Go compiler and/or runtime. label Mar 11, 2025
@cagedmantis cagedmantis self-assigned this Mar 11, 2025
@cagedmantis cagedmantis added the NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. label Mar 11, 2025
@cagedmantis
Copy link
Contributor Author

@gopherbot please backport to Go 1.24.

@gopherbot
Copy link
Contributor

Backport issue(s) opened: #72796 (for 1.24).

Remember to create the cherry-pick CL(s) as soon as the patch is submitted to master, according to https://go.dev/wiki/MinorReleases.

@cagedmantis cagedmantis added this to the Go1.25 milestone Mar 11, 2025
@cagedmantis cagedmantis added the Documentation Issues describing a change to documentation. label Mar 11, 2025
@gopherbot
Copy link
Contributor

Change https://go.dev/cl/656815 mentions this issue: [release-branch.go1.24] runtime: Added usage example for the runtime.AddCleanup() function.

@cagedmantis
Copy link
Contributor Author

Closing this issue since 649995 has already been submitted.

@dmitshur dmitshur added NeedsFix The path to resolution is known, but the work has not been done. and removed NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. labels Mar 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
compiler/runtime Issues related to the Go compiler and/or runtime. Documentation Issues describing a change to documentation. NeedsFix The path to resolution is known, but the work has not been done.
Projects
None yet
Development

No branches or pull requests

3 participants