Skip to content

ENT-14152: Reformatted with CFEngine CLI and added lint and format checks#2280

Merged
olehermanse merged 1 commit into
cfengine:masterfrom
olehermanse:master
Jun 1, 2026
Merged

ENT-14152: Reformatted with CFEngine CLI and added lint and format checks#2280
olehermanse merged 1 commit into
cfengine:masterfrom
olehermanse:master

Conversation

@olehermanse
Copy link
Copy Markdown
Member

No description provided.

Ticket: ENT-14152
Signed-off-by: Ole Herman Schumacher Elgesem <ole@northern.tech>
Copy link
Copy Markdown
Contributor

@craigcomstock craigcomstock left a comment

Choose a reason for hiding this comment

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

initially I didn't like the extra space but I suppose for ease-of-use running the formatter often, this will be good!

Comment on lines +107 to +108
debian.containers_host::
# in jenkins, CONTAINER labeled nodes are capable of running container builds like valgrind-check and static-check
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.

These lines should probably swap places

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.

I'm not really sure, it seems to refer to the class guard so makes some sense.

@olehermanse olehermanse merged commit 347fd9f into cfengine:master Jun 1, 2026
1 check 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