Skip to content

Add pipenv language support to pre-commit#1

Open
nhtgl wants to merge 10 commits intomainfrom
pipenv
Open

Add pipenv language support to pre-commit#1
nhtgl wants to merge 10 commits intomainfrom
pipenv

Conversation

@nhtgl
Copy link
Copy Markdown
Owner

@nhtgl nhtgl commented Feb 4, 2025

  • Implement pipenv language module in pre-commit
  • Update workflows to include pipenv language tests
  • Add pipenv language to allowed languages in testing script
  • Create comprehensive test suite for pipenv language support

nhtgl added 10 commits February 4, 2025 13:25
- Implement pipenv language module in pre-commit
- Update workflows to include pipenv language tests
- Add pipenv language to allowed languages in testing script
- Create comprehensive test suite for pipenv language support
- Add explicit virtualenv creation and environment settings for pipenv
- Implement more robust health check and installation processes
- Modify run_hook to use pipenv run with proper environment configuration
- Update tests to verify new pipenv language behavior
- Simplify and streamline pipenv language tests
- Enhance test coverage and readability
- Remove unnecessary imports and code complexity
- Update test fixtures and mocking strategies
- Modify test to pass script path as a single argument in pipenv run command
- Ensure compatibility with pipenv command execution in test environment
- Add tests for health check failure scenarios
- Implement tests for environment installation with and without additional dependencies
- Expand run_hook tests to cover file argument passing
- Improve test coverage for pipenv language module
…rror handling

- Add `_find_pipfile()` to search for Pipfile in current and parent directories
- Modify `_assert_pipfile_exists()` to return Pipfile directory
- Update methods to use the nearest Pipfile directory for pipenv commands
- Improve error handling in health check and environment setup
- Add comprehensive tests for Pipfile discovery and error scenarios
:
s Please enter the commit message for your changes. Lines starting
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.

1 participant