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

[ci] ignore R CMD CHECK warnings on new R version #3468

Merged
merged 2 commits into from
Oct 18, 2020

Conversation

jameslamb
Copy link
Collaborator

In CI for the R package, we check for R CMD CHECK warnings by looking for the string "warning" in its logs. That test is currently breaking builds (https://github.com/microsoft/LightGBM/runs/1268855023) for a warning that can safely be ignored:

Warning: package 'R6' was built under R version 4.0.3

This PR proposes ignoring those warnings. It also slightly simplifies the config for the UBSAN tests added in #3439 (based on #3443 (comment))

Copy link
Collaborator

@StrikerRUS StrikerRUS left a comment

Choose a reason for hiding this comment

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

Thank you!

@StrikerRUS
Copy link
Collaborator

StrikerRUS commented Oct 17, 2020

Seems that it is more complicated:

* checking whether package 'lightgbm' can be installed ... WARNING
Found the following significant warnings:
  Warning: package 'R6' was built under R version 4.0.3

@jameslamb
Copy link
Collaborator Author

Seems that it is more complicated:

* checking whether package 'lightgbm' can be installed ... WARNING
Found the following significant warnings:
  Warning: package 'R6' was built under R version 4.0.3

maybe I can turn off this specific warning with an environment variable. That would be better than allowing up to 1 warning in CI. I'll investigate

@StrikerRUS
Copy link
Collaborator

That would be better than allowing up to 1 warning in CI.

Yes, definitely!

@StrikerRUS StrikerRUS self-requested a review October 17, 2020 23:09
@jameslamb
Copy link
Collaborator Author

it looks like we can not skip this check 😢

I don't see any environment variables mentioned in the relevant code for R CMD check:

https://github.com/wch/r-source/blob/157225dfb95ea6eee615f2c34d677a0abf2015f8/src/library/base/R/library.R#L119-L150

I also tried to look for all of the environment variables that control R CMD CHECK. I'm not aware of a comprehensive list of these, but I found them by searching for Sys.getenv():

https://github.com/wch/r-source/search?q=Sys.getenv

I'm going to ask the r-package-devel mailing list. But first, to unblock our CI, in this PR I'll upgrade us to R 4.0.3. It is often 2-3 months between version upgrades (https://cran.r-project.org/bin/windows/base/old/), so that will buy us some time.

@jameslamb
Copy link
Collaborator Author

ok I've updated the versions to R 4.0.3

@jameslamb jameslamb merged commit c182555 into microsoft:master Oct 18, 2020
@jameslamb jameslamb deleted the misc/r-ci branch October 18, 2020 03:25
@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity since it was closed. To start a new related discussion, open a new issue at https://github.com/microsoft/LightGBM/issues including a reference to this.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants