Skip to content

Add module-write hook#3728

Merged
smoors merged 2 commits intoeasybuilders:developfrom
Flamefire:module-write-hook
Jun 9, 2021
Merged

Add module-write hook#3728
smoors merged 2 commits intoeasybuilders:developfrom
Flamefire:module-write-hook

Conversation

@Flamefire
Copy link
Copy Markdown
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
Copy Markdown
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.

Comment thread easybuild/tools/hooks.py Outdated
Comment thread easybuild/tools/hooks.py Outdated
Comment thread easybuild/tools/hooks.py Outdated
@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
Copy Markdown
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
Copy Markdown
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