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

avoid warning about last_warnings() #6025

Merged
merged 4 commits into from
Sep 29, 2021
Merged

Conversation

@romainfrancois
Copy link
Member Author

pillar and tibble were both using import lifecycle which creates an issue with dev rlang.

@romainfrancois
Copy link
Member Author

romainfrancois commented Sep 21, 2021

🤔 this only seems to work locally:

── R CMD check results ─────────────────────────────────── dplyr 1.0.7.9000 ────
Duration: 2m 44.3s

0 errors v | 0 warnings v | 0 notes v

R CMD check succeeded

as opposed to this with released versions:

> checking Rd files ... WARNING
  prepare_Rd: replacing previous import ‘lifecycle::last_warnings’ by ‘rlang::last_warnings’ when loading ‘pillar’
  prepare_Rd: replacing previous import ‘lifecycle::last_warnings’ by ‘rlang::last_warnings’ when loading ‘tibble’

> checking for unstated dependencies in examples ... WARNING
  Warning: replacing previous import ‘lifecycle::last_warnings’ by ‘rlang::last_warnings’ when loading ‘pillar’
  Warning: replacing previous import ‘lifecycle::last_warnings’ by ‘rlang::last_warnings’ when loading ‘tibble’

> checking Rd line widths ... NOTE
  Warning: replacing previous import ‘lifecycle::last_warnings’ by ‘rlang::last_warnings’ when loading ‘pillar’
  Warning: replacing previous import ‘lifecycle::last_warnings’ by ‘rlang::last_warnings’ when loading ‘tibble’
  These lines will be truncated in the PDF manual.

0 errors v | 2 warnings x | 1 note x
Error: R CMD check found WARNINGs
In addition: Warning messages:
1: replacing previous import ‘lifecycle::last_warnings’ by ‘rlang::last_warnings’ when loading ‘tibble’ 
2: replacing previous import ‘lifecycle::last_warnings’ by ‘rlang::last_warnings’ when loading ‘pillar’ 
Execution halted

Exited with status 1.

same with dev versions of both, i.e. r-lib/pillar and tidyverse/tibble

@lionel-
Copy link
Member

lionel- commented Sep 21, 2021

I'll also look into conditionally reexporting last_warnings() from rlang in lifecycle. I need to send a patch release soon for testthat compatibility as well.

@lionel-
Copy link
Member

lionel- commented Sep 24, 2021

lifecycle 1.0.1 just got on CRAN to fix this warning.

@krlmlr
Copy link
Member

krlmlr commented Sep 26, 2021

I'll do a patch release today, should be on CRAN tomorrow (tibble usually shows false positives in CRAN's revdep checks).

@krlmlr
Copy link
Member

krlmlr commented Sep 26, 2021

Should have read more carefully. Dedicated imports are better anyway, we're double safe here. Can we close this PR?

@krlmlr
Copy link
Member

krlmlr commented Sep 26, 2021

Thanks @lionel- for taking care of this upstream, and @romainfrancois for the pulls.

@romainfrancois
Copy link
Member Author

Merging this. The errors we see are unrelated.

@romainfrancois romainfrancois merged commit 14d5280 into master Sep 29, 2021
@romainfrancois romainfrancois deleted the last_warnings_warning branch September 29, 2021 14:10
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