Skip to content

Deprecate time_limit and limit timeout aliases in Facter::Core::Execution.execute#114

Open
silug wants to merge 1 commit into
OpenVoxProject:mainfrom
silug:deprecate-execution-timeout-aliases
Open

Deprecate time_limit and limit timeout aliases in Facter::Core::Execution.execute#114
silug wants to merge 1 commit into
OpenVoxProject:mainfrom
silug:deprecate-execution-timeout-aliases

Conversation

@silug
Copy link
Copy Markdown

@silug silug commented Apr 13, 2026

Short description

These option keys are compatibility aliases for :timeout. Emit a specific warnonce deprecation message for each when used, rather than surfacing them through the generic unexpected-key warning.

Closes #90

Checklist

I have:

  • read the CONTRIBUTING.md document
  • read and accepted the Developer Certificate of Origin document and added a Signed-off-by annotation to each of my commits
  • tested this code
  • included documentation (including possible behaviour changes)
  • documented the code
  • added or modified regression test(s)
  • added or modified unit test(s)

…tion.execute

These option keys are compatibility aliases for `:timeout`. Emit a specific
`warnonce` deprecation message for each when used, rather than surfacing
them through the generic unexpected-key warning.

Closes OpenVoxProject#90

Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
Signed-off-by: Steven Pritchard <[email protected]>
@silug silug marked this pull request as ready for review April 24, 2026 21:09
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.

[Deprecation] Legacy Timeout Option Aliases in Facter::Core::Execution.execute

1 participant