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

Requestion gsDesign2 >= v1.1.3 #300

Merged
merged 4 commits into from
Nov 18, 2024
Merged

Requestion gsDesign2 >= v1.1.3 #300

merged 4 commits into from
Nov 18, 2024

Conversation

LittleBeannie
Copy link
Collaborator

This PR follows #297.

John might be off today. @nanxstats and @yihui, could you please review and approve it? I will prepare a CRAN submission after your approval.

Copy link
Contributor

@yihui yihui left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think that's good.

@yihui
Copy link
Contributor

yihui commented Nov 15, 2024

Oh actually you might not be able to pass the automated check on CRAN because gsDesign2 1.1.3 is too new, which may not be available on CRAN's check machines yet, but you can certainly try. If it fails to pass today, you can retry the submission tomorrow and I guess it should be able to pass automatically. Feel free to ping me here if you encounter any problems during the submission.

Copy link
Collaborator

@nanxstats nanxstats left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yihui has a valid point but you'll know during CRAN submission. Also, the dependencies in DESCRIPTION are sorted alphabetically but that's only a minor style thing.

@LittleBeannie LittleBeannie merged commit bcbd880 into main Nov 18, 2024
0 of 7 checks passed
@LittleBeannie
Copy link
Collaborator Author

Thank you, @yihui, for the suggestions!

@LittleBeannie LittleBeannie deleted the 296-fix-cran-pre-tests branch November 18, 2024 03:48
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.

3 participants