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

Cran fixes #138

Merged
merged 19 commits into from
Sep 4, 2023
Merged

Cran fixes #138

merged 19 commits into from
Sep 4, 2023

Conversation

wincowgerDEV
Copy link
Owner

Opening this to start a discussion

@wincowgerDEV
Copy link
Owner Author

@zsteinmetz, went through and ignored some of the tests and examples that cran was upset about. Also tried to speed up the vignette.

@zsteinmetz
Copy link
Collaborator

Hey @wincowgerDEV, I already reduced the number of examples before. Let's try another submission with the remaining ones before excluding them completely. dontrun examples are often neglected because everybody thinks they're wrong by example.

@wincowgerDEV
Copy link
Owner Author

Sounds good!

@zsteinmetz
Copy link
Collaborator

I ran some additional tests. Timings look better now. Preparing 1.0.1 ...

@zsteinmetz zsteinmetz merged commit 8ed1ba9 into main Sep 4, 2023
7 checks passed
@zsteinmetz zsteinmetz deleted the cran-fixes branch September 4, 2023 13:59
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