Skip to content

Reformatted some initial files with cfengine format#3132

Merged
olehermanse merged 2 commits intocfengine:masterfrom
olehermanse:master
Apr 17, 2026
Merged

Reformatted some initial files with cfengine format#3132
olehermanse merged 2 commits intocfengine:masterfrom
olehermanse:master

Conversation

@olehermanse
Copy link
Copy Markdown
Member

  • Reformatted some initial files with cfengine format
  • inventory/debian.cf: Manually adjusted 1 comment location

@olehermanse olehermanse marked this pull request as ready for review April 17, 2026 11:53
Signed-off-by: Ole Herman Schumacher Elgesem <ole@northern.tech>
Signed-off-by: Ole Herman Schumacher Elgesem <ole@northern.tech>
@olehermanse
Copy link
Copy Markdown
Member Author

@cf-bottom Jenkins please :)

@cf-bottom
Copy link
Copy Markdown

Comment thread cfe_internal/CFE_cfengine.cf
Comment thread cfe_internal/CFE_cfengine.cf
Comment thread inventory/debian.cf
Comment thread inventory/debian.cf
Comment on lines +20 to +21
# array to populate
"$(linuxmint_info)",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think comments should stay on the side if that's where they where. At least, that's how black works

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Not really possible - comments are just comments and we render from syntax tree - whitespace is lost.

(We'd have to change the parser and architecture quite a bit to achieve that, and I don't think it's really worth it).

@olehermanse
Copy link
Copy Markdown
Member Author

RHEL 7 failure in Jenkins seem unrelated (connectivity issue). Acceptance tests on other platforms and GH Actions are green.

@olehermanse olehermanse merged commit 0f3f618 into cfengine:master Apr 17, 2026
12 of 13 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