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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

init commit of edits for exposing C api #69

Merged
merged 1 commit into from
Apr 23, 2018

Conversation

ChrisMuir
Copy link
Collaborator

Related to issue #67. As mentioned before, I created this branch just to get the ball rolling on the API, so am happy to make edits (or if it needs to be torn down and redone, that's cool too 馃槃 ).

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 90.229% when pulling 77dff6c on ChrisMuir:c_api into 2123513 on markvanderloo:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 90.229% when pulling 77dff6c on ChrisMuir:c_api into 2123513 on markvanderloo:master.

@markvanderloo
Copy link
Owner

Hmm can't merge because of a failing test (I'm on my phone now).

@ChrisMuir
Copy link
Collaborator Author

Hmm, all Travis tests passed, I think it's just that test coverage dropped by 0.2%. I think the drop is due to the zzz.R file I added, as the guts of .onUnload is not covered, see here.

@markvanderloo markvanderloo merged commit 6f2884d into markvanderloo:master Apr 23, 2018
@markvanderloo
Copy link
Owner

I just merged it. There was no real fail but a minor drop in the test coverage (naturally). Apparently it is not possible to merge from the phone when that happens. In any case I'm back in town and I'm able to respond much quicker. I'll have a good look at WRE on the C API this week. Looking forward to getting this out there!

@ChrisMuir ChrisMuir deleted the c_api branch April 23, 2018 13:47
@ChrisMuir
Copy link
Collaborator Author

馃憤 Sounds great!

Yeah I think the coverage drop was the zzz.R file I added, see here. No biggie.

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

3 participants