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

Add initial Travis-CI support. #5

Closed
wants to merge 8 commits into from
Closed

Conversation

oerd
Copy link
Contributor

@oerd oerd commented Jun 12, 2014

Add simple Travis-CI build support:

  • clones snowball-data from snowballstem/snowball-data
  • builds snowball
  • checks snowball via make check

oerd added 3 commits June 13, 2014 01:37
Add simple Travis-CI build support: just checks for successful builds (i.e. no `make check` test runs)
Add initial Travis-CI support.
Fix travis-ci config to get stemmer data files required during `make check`. It will recursively build snowball before checking algorithms.
@oerd oerd mentioned this pull request Jun 13, 2014
@rboulton
Copy link
Member

Thanks for this - it looks good. I need to work out some issues with merging between our git and svn repos, but as soon as this is done I'll merge this.

oerd added 5 commits July 1, 2014 22:43
* develop:
  Update .travis.yml
* whatnot:
  Revert errors on previous to compiler/driver.c from commit
  Major Java improvements: Test-phase
@rboulton rboulton mentioned this pull request Nov 9, 2014
@rboulton
Copy link
Member

rboulton commented Nov 9, 2014

I've just disabled the svn => git sync, since we're no longer planning on making changes via SVN. Therefore, I'd be happy to merge this. However, since June, there have been further changes made on this branch, so I can't simply merge this pull request. Instead, I made #9, and rebased your changes into it. Thanks very much, sorry for the slow merge.

@rboulton rboulton closed this Nov 9, 2014
ojwb added a commit that referenced this pull request Sep 4, 2015
It's the only thing blocking an explicit licence being added to the repo
(#10).  We can add it back if we hear back from the author in #5.  The
patch is perhaps too small to be subject to copyright, but for the same
reasons it wouldn't be hard to reimplement.
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.

None yet

2 participants