Skip to content

ENT-12722: Added documentation for getgroups policy function#3509

Merged
olehermanse merged 1 commit into
cfengine:masterfrom
victormlg:getgroups
Oct 6, 2025
Merged

ENT-12722: Added documentation for getgroups policy function#3509
olehermanse merged 1 commit into
cfengine:masterfrom
victormlg:getgroups

Conversation

@victormlg

@victormlg victormlg commented Sep 22, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@cf-bottom

Copy link
Copy Markdown

Thanks for submitting a pull request! Maybe @nickanderson can review this?

@nickanderson nickanderson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I am not sure that the example output is as expected given the policy.

Comment thread content/reference/functions/getgroups.markdown

@olehermanse olehermanse left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please fix the formatting by running cfengine dev docs-format.

@nickanderson nickanderson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Well, i would approve it after format update :D

@victormlg

Copy link
Copy Markdown
Contributor Author

The failing formatting checks are not related to my changes

Comment thread content/reference/functions/getgroups.markdown Outdated
Comment thread content/reference/functions/getgroups.markdown Outdated
@olehermanse

Copy link
Copy Markdown
Member

Core PR merged; cfengine/core#5880

Ticket: ENT-12722
Signed-off-by: Victor Moene <victor.moene@northern.tech>
@olehermanse

Copy link
Copy Markdown
Member

@cf-bottom jenkins, please

@cf-bottom

Copy link
Copy Markdown

@olehermanse olehermanse merged commit 655874e into cfengine:master Oct 6, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants