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

use R.cache >= 0.15.0 and avoid the prompt #819

Merged
merged 1 commit into from
Jul 7, 2021

Conversation

lorenzwalthert
Copy link
Collaborator

Closes #818.

@github-actions
Copy link
Contributor

github-actions bot commented Jul 7, 2021

This is how benchmark results would change (along with a 95% confience interval in relative change) if 9e6b4be is merged into master:

  • cache_applying: 0.1s -> 0.1s [-3.25%, +5.59%]
  • cache_recording: 1.45s -> 1.46s [-0.67%, +2.24%]
  • without_cache: 3.72s -> 3.74s [-0.38%, +1.38%]

Further explanation regarding interpretation and methodology can be found in the documentation.

@lorenzwalthert lorenzwalthert merged commit 6879d20 into r-lib:master Jul 7, 2021
@lorenzwalthert lorenzwalthert deleted the issue-818 branch July 7, 2021 18:08
lorenzwalthert added a commit to lorenzwalthert/styler that referenced this pull request Jul 18, 2021
clrpackages pushed a commit to clearlinux-pkgs/R-styler that referenced this pull request Sep 21, 2021
…1.6.1

Johannes Enevoldsen (7):
      add Rmarkdown to assertfiletype
      add Rmarkdown as accepted filetype in prettify_pgk
      add tests including and excluding Rmarkdown
      updated reference files from tests
      add news bullit about #824
      adds documentation for .Rmarkdown support
      adds missing comma in documentation

JohannesNE (1):
      Treat .Rmarkdown files as .Rmd files

Lorenz (38):
      bump devel
      don't break line around comments
      revert to old reference files
      set line break between %>% and { to 1
      set line break between %>% and { to 1
      remove R.cache message as no longer necessary with r-lib/styler#819
      allow ordinary comments in roxygen comments
      handl multiple idnetical comments right
      better functional capsuling
      document
      allow non-parsable code in headers
      two error on impossible condition
      add vs code integration
      use local testing setup
      add news
      document
      bump
      bump
      bump ci
      bump ci
      use testing fixtures
      test that cache is cleared
      test we did not use cache with tests
      must deactivate cache also for external process
      fix test
      re-document
      fix for windows
      fix touchstone script
      prepare release
      add more contributors
      comment for CRAN
      show user a message to clear old caches in interactive use only.
      make it work with no cache
      update cran comments
      remove trailing emtpy line
      Fix URLs for release 1.6.1
      add more notes
      bump devel

Lorenz Walthert (2):
      formatting
      bump version to invalidate cache.
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.

Require R.cache >= 0.15.0
1 participant