Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[blocked] Travis support #46

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

DennisSnijder
Copy link
Contributor

@DennisSnijder DennisSnijder commented Mar 25, 2019

What has been done?

Added a .travis.yml to enable travis support.

Proposed Changes

  • Added the gradlew binary for testing purposes.
  • Added a .travis.yml with the runner config
  • Requires @Tomm0017 to integrate travis with this repository through https://travis-ci.org

Notes

As you can see in the link below the test is failing. This is because the item container test needs the actual cache. @Tomm0017 is working on mocking the tests so it actually works without using a cache 😄.

https://travis-ci.org/DennisSnijder/rsmod/jobs/511200812

blocked by #53

.gitignore Outdated
@@ -3,7 +3,6 @@
build/
out/

gradlew
gradlew.bat
Copy link
Contributor

Choose a reason for hiding this comment

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

Should probably commit gradle.bat also so windows isn't left out with being able to have a gradle wrapper available.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fair point! fixing this 😄

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.

2 participants