Skip to content

Conversation

jakub-nt
Copy link
Contributor

Ticket: ENT-12740

Changelog: None

@jakub-nt jakub-nt marked this pull request as ready for review September 29, 2025 17:54
services/autorun/README.md
templates/README.md
ci
modules/mustache
Copy link
Contributor Author

@jakub-nt jakub-nt Sep 29, 2025

Choose a reason for hiding this comment

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

Not 100% sure about this one (deleting the entire mustache directory).

Copy link
Member

Choose a reason for hiding this comment

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

It's empty (only a README), and it's also not present in the tarballs from the website. I think it makes sense to remove here to make the module minimal, and to make it match the tarball. It's the type of thing which should be documented elsewhere (create a folder / file or use this module, or similar).

@olehermanse olehermanse merged commit c91eb66 into cfengine:master Sep 30, 2025
4 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