GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — pre-commit does not work when used in a project with R renv · Issue #3385 · pre-commit/pre-commit

pre-commit does not work when used in a project with R renv #3385

New issue
Copy link
New issue
Copy link
Closed
#3389
Closed
pre-commit does not work when used in a project with R renv#3385
#3389
Copy link

Description

@lorenzwalthert
lorenzwalthert
opened on Jan 10, 2025
Issue body actions

search you tried in the issue tracker

r, renv

describe your issue

As of now, the environment variables $RENV_PROJECT and $R_PROFILE_USER are not unset during the installation process, so the hooks won't be properly installed in an R project that uses renv in an IDE like RStudio where these env variables were set already. During hook execution, the hook patch is already applied correctly.

This problem (with a reprex) was reported downstream.

I will go ahead and submit a PR to fix it, adapting the designated wrapper to call R should suffice. Hope it's ok to not provide a .pre-commit-config.yaml for this.

pre-commit --version

pre-commit 4.0.1

.pre-commit-config.yaml

Not needed

~/.cache/pre-commit/pre-commit.log (if present)

No response

Activity

  1. asottile
    closed this as completedin #3389on Jan 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions

      You are visitor 01402503 — GeezerVison does not store the pages you look up.
      Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.