Skip to content

ENT-13073: Added linter errors for unknown promise types#30

Merged
olehermanse merged 1 commit intocfengine:mainfrom
SimonThalvorsen:ENT-13073
Mar 27, 2026
Merged

ENT-13073: Added linter errors for unknown promise types#30
olehermanse merged 1 commit intocfengine:mainfrom
SimonThalvorsen:ENT-13073

Conversation

@SimonThalvorsen
Copy link
Copy Markdown
Contributor

No description provided.

@SimonThalvorsen SimonThalvorsen marked this pull request as draft March 23, 2026 11:53
@SimonThalvorsen SimonThalvorsen force-pushed the ENT-13073 branch 4 times, most recently from 8584d82 to d3c5fe5 Compare March 23, 2026 13:07
@SimonThalvorsen SimonThalvorsen marked this pull request as ready for review March 23, 2026 13:11
@SimonThalvorsen
Copy link
Copy Markdown
Contributor Author

@olehermanse I am unable to request reviewers

@larsewi larsewi requested a review from olehermanse March 25, 2026 13:55
Copy link
Copy Markdown
Member

@olehermanse olehermanse left a comment

Choose a reason for hiding this comment

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

Looks good overall, let's get rid of the global mutable variables (consts are fine).

@SimonThalvorsen SimonThalvorsen force-pushed the ENT-13073 branch 2 times, most recently from 5976b3c to 905afe4 Compare March 27, 2026 13:30
Signed-off-by: Simon Halvorsen <simon.halvorsen@northern.tech>
@olehermanse olehermanse changed the title ENT-13073: Added linter errors for unknown promisetypes ENT-13073: Added linter errors for unknown promise types Mar 27, 2026
@olehermanse olehermanse merged commit 01fdec6 into cfengine:main Mar 27, 2026
6 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.

2 participants