Skip to content

deprecation: add runtime warnings to deprecated exec/which methods#121

Draft
silug wants to merge 1 commit intoOpenVoxProject:mainfrom
silug:deprecation-warnings-exec-which
Draft

deprecation: add runtime warnings to deprecated exec/which methods#121
silug wants to merge 1 commit intoOpenVoxProject:mainfrom
silug:deprecation-warnings-exec-which

Conversation

@silug
Copy link
Copy Markdown

@silug silug commented Apr 24, 2026

Short description

Add explicit runtime deprecation warnings (via Facter.warnonce) to methods that are slated for removal in a future major version:

  • Facter::Core::Execution.exec: warn callers to use .execute instead
  • Facter::Util::Resolution.exec: replaced bare public :exec with an explicit method that warns and delegates to Facter::Core::Execution
  • Facter::Util::Resolution.which: same treatment as .exec above

Specs updated to assert the deprecation warning is emitted; new specs added for the Resolution class-level deprecated methods.

References #88, #89

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)

Add explicit runtime deprecation warnings (via Facter.warnonce) to
methods that are slated for removal in a future major version:

- Facter::Core::Execution.exec: warn callers to use .execute instead
- Facter::Util::Resolution.exec: replaced bare public :exec with an
  explicit method that warns and delegates to Facter::Core::Execution
- Facter::Util::Resolution.which: same treatment as .exec above

Specs updated to assert the deprecation warning is emitted; new specs
added for the Resolution class-level deprecated methods.

References OpenVoxProject#88, OpenVoxProject#89
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.

1 participant