Skip to content

Conversation

@boegel
Copy link
Member

@boegel boegel commented Aug 9, 2023

No description provided.

@boegel boegel added this to the 5.0 milestone Aug 9, 2023
@easybuilders easybuilders deleted a comment from boegelbot Aug 11, 2023
Copy link
Member

@ocaisa ocaisa 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, just needs the changes @branfosj suggested

ocaisa and others added 2 commits August 11, 2023 15:52
@ocaisa
Copy link
Member

ocaisa commented Aug 11, 2023

I committed the changes from @branfosj as they are trivial and consistent with previous usage

@ocaisa ocaisa dismissed branfosj’s stale review August 11, 2023 14:23

Requested changes have been merged

@ocaisa ocaisa merged commit 629b312 into easybuilders:5.0.x Aug 11, 2023
@boegel boegel deleted the run_modules branch August 11, 2023 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

No open projects
Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants