Skip to content

Conversation

dannystaple
Copy link
Contributor

Only a tiny one, but it's not uncommon on a dev setup to have a .venv folder, which these checks will try to run on.

@dannystaple dannystaple changed the title Switch venv folders Exclude venv folders from checks Jul 8, 2025
@Gadgetoid
Copy link
Member

Interesting, I use one big hot mess of a "dev" venv but ratifying this approach as part of the boiletplate - make dev - could be useful.

Totally haven't had builds/commands fail because I didn't have the right packages installed in whatever venv I happened to be using at the time, and then just run make dev-deps into that random venv nooo.

@dannystaple
Copy link
Contributor Author

Interesting, I use one big hot mess of a "dev" venv but ratifying this approach as part of the boiletplate - make dev - could be useful.

Totally haven't had builds/commands fail because I didn't have the right packages installed in whatever venv I happened to be using at the time, and then just run make dev-deps into that random venv nooo.

I've been migrating stuff I've been using to uv, having previously used poetry before. It is starting to lead to much tidier venvs - but I used to use 1 or 2 venvs for many projects too. That gets harder as the number of projects I'm juggling increases so I started to mostly say 1 repo gets 1 venv.

@Gadgetoid
Copy link
Member

I've been meaning to check out uv for a while. I'll get 'round to it... one day...

@coveralls
Copy link

Pull Request Test Coverage Report for Build 16140299768

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 91.667%

Totals Coverage Status
Change from base Build 11123021070: 0.0%
Covered Lines: 11
Relevant Lines: 12

💛 - Coveralls

@Gadgetoid Gadgetoid merged commit 2041697 into pimoroni:main Jul 8, 2025
10 checks passed
@dannystaple dannystaple deleted the exclude-venv-folders branch July 8, 2025 13:23
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.

3 participants