Skip to content

Ref #214 -- Set language to automatically install dependency - #381

Closed
codingjoe wants to merge 1 commit into
seddonym:mainfrom
codingjoe:patch-1
Closed

codingjoe wants to merge 1 commit into
seddonym:mainfrom
codingjoe:patch-1

Conversation

@codingjoe

@codingjoe codingjoe commented Sep 2, 2026 •

Copy link
Copy Markdown

Pre-commit will automatically install dependencies (in a separate env) if you tell it to. Which you do via the language.

Ref #214

  • Add tests for the change.
  • Add any appropriate documentation.
  • Run just check.
  • Add a summary of changes to docs/release_notes.md.
  • Add your name to docs/authors.md (in alphabetical order).

Pre-commit will automatically install dependencies (in a separate env) if you tell it to. Which you do via the language. 

Ref seddonym#214
@seddonym

seddonym commented Sep 3, 2026

Copy link
Copy Markdown
Owner

Thanks, but this won't work as it needs to install the Python packages under test within the virtual environment for Import Linter to work. From the docs.

It is possible to run Import Linter as a pre-commit hook. However, this must use language: system to allow Import Linter to analyze your packages from within a virtual environment.

More discussion on #268 and the PRs it links to.

@seddonym seddonym closed this Sep 3, 2026
@codingjoe
codingjoe deleted the patch-1 branch September 4, 2026 16:18
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.

2 participants