dosubot[bot] commented on PR #36315: URL: https://github.com/apache/superset/pull/36315#issuecomment-3604058881
<!-- Answer --> You can run pre-commit hooks inside the Superset Docker container, and this is a reliable workaround for Windows/WSL environments where native setup is problematic. The dev image includes all necessary dependencies and mounts your local repo, so any formatting fixes or changes will be reflected in your files. To do this, start your containers with `docker compose up --build`, then run: ```bash docker compose exec superset_app bash pre-commit run --all-files ``` Or, for a one-off command: ```bash docker compose run --rm superset pre-commit run --all-files ``` This approach works on any OS and is functionally equivalent to running pre-commit on Mac or Linux. If you hit issues, make sure your repo is mounted correctly and the dev dependencies are installed in the container. For more details, see the [Superset development docs](https://github.com/apache/superset/blob/829e4d92d91ceae4b43b1ed3b063ffe45377799c/docs/docs/contributing/development.mdx#L12-L505). <!-- Dosu Comment Footer --> *To reply, just mention [@dosu](https://go.dosu.dev/dosubot).* --- <sup>How did I do? [Good](https://app.dosu.dev/response-feedback/3eba836d-f79e-48a9-8bda-4381cbe74c48?feedback_type=great_response) | [Irrelevant](https://app.dosu.dev/response-feedback/3eba836d-f79e-48a9-8bda-4381cbe74c48?feedback_type=irrelevant_answer) | [Incorrect](https://app.dosu.dev/response-feedback/3eba836d-f79e-48a9-8bda-4381cbe74c48?feedback_type=incorrect_sources) | [Verbose](https://app.dosu.dev/response-feedback/3eba836d-f79e-48a9-8bda-4381cbe74c48?feedback_type=too_verbose) | [Hallucination](https://app.dosu.dev/response-feedback/3eba836d-f79e-48a9-8bda-4381cbe74c48?feedback_type=hallucination) | [Report 🐛](https://app.dosu.dev/response-feedback/3eba836d-f79e-48a9-8bda-4381cbe74c48?feedback_type=bug_report) | [Other](https://app.dosu.dev/response-feedback/3eba836d-f79e-48a9-8bda-4381cbe74c48?feedback_type=other)</sup> [](https://app.dosu.dev/a28d3c7e-a9d3-459e-9fb6-3a6f9ff4f357/ask?utm_source=github) [](https://cursor.com/link/prompt?text=You%20can%20run%20pre-commit%20hooks%20inside%20the%20Superset%20Docker%20container%2C%20and%20this%20is%20a%20reliable%20workaround%20for%20Windows/WSL%20environments%20where%20native%20setup%20is%20problematic.%20The%20dev%20image%20includes%20all%20necessary%20dependencies%20and%20mounts%20your%20local%20repo%2C%20so%20any%20formatting%20fixes%20or%20changes%20will%20be%20reflected%20in%20your%20files.%0A%0ATo%20do%20this%2C%20start%20your%20containers%20with%20%60docker%20compose%20up%20--build%60%2C%20then%20run%3A%0A%0A%60%60%60bash%0Adocker%20compose%20exec%20superset_app%20bash%0Apre-commit%20run%20--all-files%0A%60%60%60%0A%0AOr%2C%20for%20a%20one-off%20command%3A%0A%0A%60%60%60bash%0Adocker%20compose%20run%20--rm%20superset%20pre-commit%2 0run%20--all-files%0A%60%60%60%0A%0AThis%20approach%20works%20on%20any%20OS%20and%20is%20functionally%20equivalent%20to%20running%20pre-commit%20on%20Mac%20or%20Linux.%20If%20you%20hit%20issues%2C%20make%20sure%20your%20repo%20is%20mounted%20correctly%20and%20the%20dev%20dependencies%20are%20installed%20in%20the%20container.%20For%20more%20details%2C%20see%20the%20%5BSuperset%20development%20docs%5D%28https%3A//github.com/apache/superset/blob/829e4d92d91ceae4b43b1ed3b063ffe45377799c/docs/docs/contributing/development.mdx%23L12-L505%29.) [](https://go.dosu.dev/discord-bot) [](https://twitter.com/intent/tweet?text=%40dosu_ai%20helped%20me%20solve%20this%20issue!&url=https%3A//github.com/apache/superset/pull/36315) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
