Skip to content

Conversation

@Flamefire
Copy link
Contributor

This hook will be called anytime a module is written, so even for the fake modules used for sanity checks etc.
It gets passed the easyblock as the first argument and the filename and content to be written as the next 2 and may return a new text to be used for the module file. If nothing or None is returned the original text will be used.

This hook will be called anytime a module is written, so even for the
fake modules used for sanity checks etc.
It gets passed the easyblock as the first argument and the filename and
content to be written as the next 2 and may return a new text to be used
for the module file. If nothing or None is returned the original text
will be used.
Copy link
Contributor

@smoors smoors 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 to me, apart from some comment updates.

@Flamefire Flamefire requested a review from smoors June 9, 2021 11:09
@smoors smoors added this to the next release (4.4.1) milestone Jun 9, 2021
Copy link
Contributor

@smoors smoors left a comment

Choose a reason for hiding this comment

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

lgtm

@smoors
Copy link
Contributor

smoors commented Jun 9, 2021

Going in, thanks @Flamefire!

@smoors smoors merged commit a80f589 into easybuilders:develop Jun 9, 2021
@Flamefire Flamefire deleted the module-write-hook branch June 9, 2021 11:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants