Skip to content

add test configuration for Travis#1733

Merged
boegel merged 24 commits intoeasybuilders:developfrom
boegel:travis
Apr 22, 2016
Merged

add test configuration for Travis#1733
boegel merged 24 commits intoeasybuilders:developfrom
boegel:travis

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Apr 22, 2016

No description provided.

boegel added 23 commits April 20, 2016 17:30
…directly is more troublesome because of vsc-install
having 'lmod' available in $PATH is still required for the bootstrap script

This reverts commit 2ea7bf2.
@boegel boegel added this to the v2.8.0 milestone Apr 22, 2016
@boegel
Copy link
Copy Markdown
Member Author

boegel commented Apr 22, 2016

@wpoely86 please review? ^_^

Comment thread .travis.yml
# for recent versions of keyring, keyrings.alt must be installed too
- if [ "x$TRAVIS_PYTHON_VERSION" == 'x2.6' ]; then pip install keyring==5.7.1; else pip install keyring keyrings.alt; fi
# optional Python packages for EasyBuild
- pip install autopep8 GC3Pie GitPython python-graph-dot python-hglib PyYAML
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

hg also as package? And pep8 itself for the style tests

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

seems like hg/Mercurial is just there?

we will add pep8 when we actually need it

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I included mercurial in the list of packages to install above.

@wpoely86
Copy link
Copy Markdown
Member

Looks great

@hpcugentbot
Copy link
Copy Markdown

EasyBuild framework unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-framework-pr-builder/2956/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.

@hpcugentbot
Copy link
Copy Markdown

EasyBuild framework unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-framework-pr-builder/2957/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 Apr 22, 2016

Going in, thanks for the review @wpoely86!

I'll leave both Jenkins and Travis enabled for a while, before we switch to Travis-only...

@boegel boegel merged commit 15bced3 into easybuilders:develop Apr 22, 2016
@boegel boegel deleted the travis branch April 22, 2016 16:37
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