Skip to content

Conversation

@imaqsood
Copy link

Summary

This updates the user parameter description to clarify the behavior of ensure => 'absent', explicitly stating that the user, their home directory, and any managed SSH keys are fully removed regardless of purge_home_user or purge_sshkeys settings. Also improves wording consistency and readability.

Checklist

  • 🟢 Spec tests.
  • 🟢 Acceptance tests.
  • Manually verified. (For example puppet apply)

…tion

This updates the user parameter description to clarify the behavior of
`ensure => 'absent'`, explicitly stating that the user, their home
directory, and any managed SSH keys are fully removed regardless of
`purge_home_user` or `purge_sshkeys` settings. Also improves wording
consistency and readability.
@imaqsood imaqsood requested review from a team and bastelfreak as code owners June 23, 2025 11:33
@imaqsood imaqsood requested a review from Copilot June 23, 2025 16:45
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR clarifies the behavior of ensure => 'absent' in the accounts::user reference, explicitly stating that the user, their home directory, and managed SSH keys are fully removed regardless of related purge settings. It also improves wording consistency and readability.

  • Specifies behavior for both 'present' and 'absent' cases.
  • Updates terminology for home directory and SSH keys.
  • Notes override of purge_home_user and purge_sshkeys.
Comments suppressed due to low confidence (1)

REFERENCE.md:198

  • [nitpick] Wrap the values 'present' and 'absent' in inline code formatting (e.g., present and absent) to maintain consistency with other parameter descriptions and default value styling.
should exist. Valid values are 'present' and 'absent'. When set to 'present',

@imaqsood imaqsood closed this Aug 6, 2025
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