Skip to content

check whether dependencies marked as external module are hidden#1552

Merged
boegel merged 2 commits intoeasybuilders:developfrom
boegel:external_hidden
Jan 20, 2016
Merged

check whether dependencies marked as external module are hidden#1552
boegel merged 2 commits intoeasybuilders:developfrom
boegel:external_hidden

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Jan 17, 2016

fixes issue reported in easybuilders/easybuild#185

@DirkdeDraak: can you check whether the problem you reported is fixed with this patch (which is a better place to handle it than the patch you proposed imho)?

@hpcugentbot
Copy link
Copy Markdown

EasyBuild framework unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-framework-pr-builder/2516/console for more details).

This pull request is now ready for review/testing.

Please try and find someone who can tackle this; contact @boegel if you're not sure what to do.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Jan 18, 2016

@wpoely86: please review?

@wpoely86
Copy link
Copy Markdown
Member

lgtm

@wpoely86
Copy link
Copy Markdown
Member

Maybe add test? 😈

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Jan 20, 2016

@wpoely86: unit test (case) added

@hpcugentbot
Copy link
Copy Markdown

EasyBuild framework unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-framework-pr-builder/2526/console for more details).

This pull request is now ready for review/testing.

Please try and find someone who can tackle this; contact @boegel if you're not sure what to do.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Jan 20, 2016

Thanks for the review @wpoely86!

@DirkdeDraak: do keep us posted whether this solves the problem you reported

boegel added a commit that referenced this pull request Jan 20, 2016
check whether dependencies marked as external module are hidden
@boegel boegel merged commit 46d6faa into easybuilders:develop Jan 20, 2016
@boegel boegel deleted the external_hidden branch January 20, 2016 12:34
@boegel boegel mentioned this pull request Jan 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants