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

Enable Travis to run on multiple OS #96

Merged

Conversation

jvcoutinho
Copy link
Collaborator

@jvcoutinho jvcoutinho commented Aug 16, 2019

Closes #68.

  • Besides Linux, Travis now run on Windows and MacOS.

    • Travis' support for Windows is still in beta. Temporarily, the three are running in a plain bash environment instead of the Java environment (supported on the other two only).
  • Shell tests haven't been rewritten as JUnit tests, but I've anticipated a refactoring and improvement work.

  • Both binaries (along with git lfs) have been excluded from the project. This means one cannot use the installer when this PR is merged, but it should last until the new installer is ready.

Sorry for the huge number of changed files. The only files relevant are:
.travis.yml,
build.gradle and
s3mGitIntegrationTests.sh

The rest are mostly test files and dependencies.

@jvcoutinho jvcoutinho force-pushed the multiple-os branch 4 times, most recently from ecd920c to 20caf3c Compare August 23, 2019 22:43
@guilhermejccavalcanti
Copy link
Owner

@jvcoutinho esse pull request ainda é relavante/necessário tendo em vista a migração desses testes para o instalador?

@jvcoutinho
Copy link
Collaborator Author

@guilhermejccavalcanti Sim, pois o PR seta o Travis para rodar em múltiplos SOs. Ainda sim, achei melhor deixar o Travis testar os testes de sistema (os "de shell", agora em JUnit) porque é uma garantia que os testes do instalador vão funcionar também.

@guilhermejccavalcanti
Copy link
Owner

@jvcoutinho beleza. verifica o conflito acima, deve ser sido por conta dos outros prs que integrei.

@jvcoutinho
Copy link
Collaborator Author

@guilhermejccavalcanti done

@guilhermejccavalcanti guilhermejccavalcanti merged commit 1ffe79d into guilhermejccavalcanti:master Sep 23, 2019
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.

Travis build considering different OS
2 participants