Skip to content

Rewrite of Virtual Environment Episode using PDM - #539

Merged
bielsnohr merged 8 commits into
devfrom
bielsnohr/venv-rewrite
Nov 12, 2026
Merged

bielsnohr merged 8 commits into
devfrom
bielsnohr/venv-rewrite

Conversation

@bielsnohr

@bielsnohr bielsnohr commented Aug 25, 2026 •

Copy link
Copy Markdown
Collaborator

This is a complete draft of changes to 1.2 Virtual Environment using PDM instead of the current pip + venv. This does not yet look at what the ripple effects would be across the rest of the lesson, which is why this merge would be into the dev branch, after which the other work on updating the remainder of the lesson would take place. This is solely to assess what a PDM-based lesson would look like.

@github-actions

github-actions Bot commented Aug 25, 2026 •

Copy link
Copy Markdown

🆗 Pre-flight checks passed 😃

This pull request has been checked and contains no modified workflow files, spoofing, or invalid commits.

Results of any additional workflows will appear here when they are done.

github-actions Bot pushed a commit that referenced this pull request Aug 25, 2026
github-actions Bot pushed a commit that referenced this pull request Sep 3, 2026
@bielsnohr bielsnohr changed the title Preview of Environment Rewrite (first-half of episode) Rewrite of Virtual Environment Episode using PDM Sep 3, 2026
Comment thread episodes/12-virtual-environments.md Resolved
Comment thread episodes/12-virtual-environments.md Resolved

@douglowe douglowe left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've gone over section 1 and made some suggested changes. We can discuss these in the meeting tomorrow.

Comment thread episodes/12-virtual-environments.md Resolved
Comment thread episodes/12-virtual-environments.md Resolved
Comment thread episodes/12-virtual-environments.md Resolved
Comment thread episodes/12-virtual-environments.md Resolved
Comment thread episodes/12-virtual-environments.md Resolved
Comment thread episodes/12-virtual-environments.md Resolved
Comment thread episodes/12-virtual-environments.md Resolved
Comment thread episodes/12-virtual-environments.md Resolved
Comment thread episodes/12-virtual-environments.md Resolved
Comment thread episodes/12-virtual-environments.md Resolved
Comment thread episodes/12-virtual-environments.md Resolved
Comment thread episodes/12-virtual-environments.md Resolved
Comment thread episodes/12-virtual-environments.md Resolved
Comment thread episodes/12-virtual-environments.md Resolved
Comment thread episodes/12-virtual-environments.md Resolved
@douglowe

Copy link
Copy Markdown
Collaborator

I've gone through all the changes here now. It is looking good, most of my suggested changes are stylistic rather than proper content changes.

Co-authored-by: Douglas Lowe <10961945+douglowe@users.noreply.github.com>

@bielsnohr bielsnohr left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the review @douglowe! I've applied all of your suggestions except for the one about using uv and pdm with conda packages. Otherwise, these were helpful and improve this episode.

Comment thread episodes/12-virtual-environments.md Resolved
Comment thread episodes/12-virtual-environments.md Resolved
Comment thread episodes/12-virtual-environments.md Resolved
Comment thread episodes/12-virtual-environments.md Resolved
Comment thread episodes/12-virtual-environments.md Resolved
Comment thread episodes/12-virtual-environments.md Resolved
Comment thread episodes/12-virtual-environments.md Resolved
Comment thread episodes/12-virtual-environments.md Resolved
Comment thread episodes/12-virtual-environments.md Resolved
github-actions Bot pushed a commit that referenced this pull request Nov 12, 2026
Comment thread episodes/12-virtual-environments.md Resolved
github-actions Bot pushed a commit that referenced this pull request Nov 12, 2026
@bielsnohr
bielsnohr merged commit 14cd4bc into dev Nov 12, 2026
4 checks passed
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