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

PR - develop-travis to develop #26

Merged
merged 30 commits into from
May 3, 2017
Merged

PR - develop-travis to develop #26

merged 30 commits into from
May 3, 2017

Conversation

timtrice
Copy link
Contributor

@timtrice timtrice commented May 3, 2017

This branch does not include tests or vignettes which at the moment fail in travis (see Issue #23). This current version passes and is ready to be merged.

develop-travis is to be locked to avoid deletion. Vignettes and tests are to be reintroduced and examined to meet whatever qualifications necessary to pass. Those have been created in Issues #24 and Issues #25. The original Issue, #23, has been closed with this PR.

timtrice added 30 commits May 2, 2017 16:42
For my latest magic trick I'll prevent this build from passing
travis. I would prefer to have the vignettes tested but at
this point, whatever.
Copied silently from github bozenne/butils.base
The most recent travis build that passed used this file:
https://goo.gl/zM6RO7. I have also removed vignettes and after
this push will await my fate.
This reverts commit e9ce8ff.
The most recent successful travis test, https://goo.gl/XMhlnK, did
not have xml2 package in the DESCRIPTION file. This test passed
though because in the .travis.yml file the warnings_are_errors
param is set to false.

When running a R CMD check a warning is generated:

```
checking dependencies in R code ... WARNING
'::' or ':::' import not declared from: ‘xml2’
```

So, first I'll see if this tests passes. Then, if so, will address
fixing the check issue.
The previous push to travis failed (Build 35). I have added xml2
back to Imports. removed getting-started.R from inst/docs and
some cosmetic changes.
@timtrice timtrice added the In Progress Actively under development label May 3, 2017
@timtrice timtrice merged commit c95bbef into develop May 3, 2017
@timtrice timtrice removed the In Progress Actively under development label May 3, 2017
@timtrice timtrice deleted the develop-travis branch May 5, 2017 12:33
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.

1 participant