Skip to content

CFE-4512: Added documentation for getgroupinfo policy function#3510

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

CFE-4512: Added documentation for getgroupinfo policy function#3510
olehermanse merged 1 commit into
cfengine:masterfrom
victormlg:getgroupinfo

Conversation

@victormlg

@victormlg victormlg commented Sep 22, 2025 •
edited by olehermanse
Loading

Copy link
Copy Markdown
Contributor

No description provided.

@cf-bottom

cf-bottom commented Sep 23, 2025

Copy link
Copy Markdown

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

craigcomstock
craigcomstock approved these changes Sep 23, 2025
View reviewed changes
Comment thread content/reference/functions/getgroupinfo.markdown Outdated
olehermanse
olehermanse requested changes Sep 30, 2025
View reviewed changes

@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.

@victormlg

victormlg commented Oct 2, 2025

Copy link
Copy Markdown
Contributor Author

The failing formatting checks are not related to my changes

@victormlg victormlg requested a review from olehermanse October 2, 2025 12:37
olehermanse
olehermanse reviewed Oct 3, 2025
View reviewed changes
Comment thread content/reference/functions/getgroupinfo.markdown Outdated
olehermanse
olehermanse reviewed Oct 3, 2025
View reviewed changes
Comment thread content/reference/functions/getgroupinfo.markdown
title: getgroupinfo
---

{{< CFEngine_function_prototype(optional_gidorname) >}}

@olehermanse olehermanse Oct 3, 2025

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.

Do we write optional_ like this in other functions?

Suggested change
{{< CFEngine_function_prototype(optional_gidorname) >}}
{{< CFEngine_function_prototype(optional_gidorname) >}}

@victormlg victormlg Oct 6, 2025

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes, we do. For example: https://docs.cfengine.com/docs/master/reference/functions/getbundlemetatags/#top

olehermanse
olehermanse reviewed Oct 3, 2025
View reviewed changes
Comment thread content/reference/functions/getgroupinfo.markdown Outdated
olehermanse
olehermanse requested changes Oct 3, 2025
View reviewed changes

@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.

See comments above.

@olehermanse

olehermanse commented Oct 3, 2025

Copy link
Copy Markdown
Member

Core PR merged: cfengine/core#5881

@victormlg
Added documentation for getgroupinfo policy function
ed77ce6 
Ticket: CFE-4512
Signed-off-by: Victor Moene <victor.moene@northern.tech>
@victormlg victormlg requested a review from olehermanse October 6, 2025 07:34
@olehermanse

olehermanse commented Oct 6, 2025

Copy link
Copy Markdown
Member

@cf-bottom jenkins, please

@cf-bottom

cf-bottom commented Oct 6, 2025

Copy link
Copy Markdown

Alright, I triggered a build:

Build Status

Jenkins: https://ci.cfengine.com/job/fast-build-and-deploy-docs-master/193/

Documentation: http://buildcache.cfengine.com/packages/build-documentation-pr/jenkins-fast-build-and-deploy-docs-master-193/output/_site/

@olehermanse olehermanse merged commit 6d5dfb6 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

Reviewers

@craigcomstock craigcomstock craigcomstock approved these changes
@olehermanse olehermanse olehermanse approved these changes
@nickanderson nickanderson Awaiting requested review from nickanderson

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@victormlg @cf-bottom @olehermanse @craigcomstock