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

Update CRAN version of shinyAce #35

Merged
merged 8 commits into from
Feb 4, 2016
Merged

Update CRAN version of shinyAce #35

merged 8 commits into from
Feb 4, 2016

Conversation

vnijs
Copy link
Collaborator

@vnijs vnijs commented Feb 4, 2016

This PR attempts to address the concerns listed in issue #26. The remaining notes from R-check (3.2.3) are:

  • checking installed package size ... NOTE
    installed size is 9.9Mb
    sub-directories of 1Mb or more:
    www 9.7Mb
  • checking dependencies in R code ... NOTE
    Unexported object imported by a ':::' call: ‘utils:::.win32consoleCompletion’
    See the note in ?::: about the use of this operator.

@trestletech
Copy link
Owner

Thanks so much for looking at this!

I just submitted a PR to your branch that will clean up a few remaining loose ends and will hopefully be good enough to get back onto CRAN.

Let me know if the stacked PR thing causes problems. If so I can just close this and merge my branch that includes these fixes.

Clean up remaining warning
trestletech added a commit that referenced this pull request Feb 4, 2016
Update CRAN version of shinyAce
@trestletech trestletech merged commit 368322a into trestletech:master Feb 4, 2016
@trestletech
Copy link
Owner

I'm going to let this soak for a couple of days to make sure nobody reports any show-stopping issues. But if you haven't heard back from me by early next week, feel free to ping me on this again and we'll try to get it up on CRAN next week.

Thanks again!

@vnijs
Copy link
Collaborator Author

vnijs commented Feb 4, 2016

Excellent! Thanks Jeff.

@vnijs
Copy link
Collaborator Author

vnijs commented Mar 3, 2016

You said to ping you so ... ping. Any updates?

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.

2 participants