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

More enums #54

Merged
merged 4 commits into from May 29, 2016
Merged

More enums #54

merged 4 commits into from May 29, 2016

Conversation

@thrasibule
Copy link
Contributor

@thrasibule thrasibule commented May 18, 2016

This closes #53 and closes #52.

@thrasibule thrasibule force-pushed the thrasibule:more_enums branch from 101a9a3 to 5db50d2 May 18, 2016
@eddelbuettel
Copy link
Owner

@eddelbuettel eddelbuettel commented May 19, 2016

Looks like a clean patch. Does R CMD check work for you locally?

To merge it would be much preferable to have the test pass...

@thrasibule
Copy link
Contributor Author

@thrasibule thrasibule commented May 19, 2016

Yes it does. I have disabled one test let's see what happens.

@thrasibule thrasibule force-pushed the thrasibule:more_enums branch 3 times, most recently from cd058f7 to 0ab842e May 19, 2016
@thrasibule thrasibule force-pushed the thrasibule:more_enums branch from 0ab842e to bdd6f91 May 27, 2016
@eddelbuettel
Copy link
Owner

@eddelbuettel eddelbuettel commented May 29, 2016

This looks good; merging now.

@eddelbuettel eddelbuettel merged commit 0190b52 into eddelbuettel:master May 29, 2016
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@thrasibule thrasibule deleted the thrasibule:more_enums branch Jun 2, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

2 participants
You can’t perform that action at this time.