Skip to content

Change project dependency manager from poetry to pdm/uv#269

Merged
Ingerid merged 18 commits intomainfrom
poetry-to-pdm
Sep 24, 2025
Merged

Change project dependency manager from poetry to pdm/uv#269
Ingerid merged 18 commits intomainfrom
poetry-to-pdm

Conversation

@Ingerid
Copy link
Copy Markdown
Collaborator

@Ingerid Ingerid commented Sep 24, 2025

Endringer

  • Oppdater prosjektversjonen til 2.42.0 i pyproject.toml
  • Legg til nye konfigurasjonsblokker i pyproject.toml som er agnostiske mht. uv, poetry eller pdm.
  • Fjern alle gamle [tool.poetry]-blokker
  • Slå sammen dependency groups dev og test, og fjern mypy, pandas-stubs og numpy fra depedencies. Numpy installeres uansett via pandas.
  • Fjern spesifikke versjoner fra dependencies, slik at pakkeversjonene kan løses ulikt for ulike pythonversjoner (gjelder særlig python 3.10-3.11 for flere pakker, og numpy 1.2-2.0 for python 3.13)
  • Bruk uv i CI.yml (github actions) istedet for poetry
  • Legg til python 3.13 i testregimet slik at dhlab dekker alle offisielt "stabile" python-versjoner
  • Oppdater til å bruke uv i release.yml også.
  • ignorer lock-filer

Closes #232

@Ingerid Ingerid requested review from Copilot and magbb September 24, 2025 11:48
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR migrates the project's dependency management from Poetry to PDM/uv by introducing standardized PEP 621 configuration while maintaining backward compatibility. The changes include updating CI workflows to use PDM and bumping the project version to 2.42.0.

  • Adds PEP 621 compliant [project] section in pyproject.toml for dependency manager agnostic configuration
  • Updates GitHub Actions workflow to use PDM instead of Poetry for dependency installation
  • Cleans up duplicate dependencies and removes unused development dependencies

Reviewed Changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.

File Description
pyproject.toml Adds PEP 621 project configuration, updates version to 2.42.0, and maintains Poetry sections for compatibility
.github/workflows/CI.yml Replaces Poetry installation and caching with PDM setup for CI pipeline
Comments suppressed due to low confidence (1)

pyproject.toml:107

  • The Poetry test dependencies section contains duplicated dependencies that are already defined in the new PEP 621 test dependency group (lines 65-69). This creates maintenance overhead and potential version conflicts. Consider removing this Poetry section or ensuring version consistency.
[tool.poetry.group.test.dependencies]
mypy = "^1.6.1"
pytest = "^7.4.3"
pytest-cov = "^4.1.0"

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread pyproject.toml Outdated
@Ingerid Ingerid requested a review from Copilot September 24, 2025 14:33
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 6 out of 7 changed files in this pull request and generated 3 comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread .github/workflows/CI.yml
Comment thread .github/workflows/release.yml
Comment thread .github/workflows/release.yml
@Ingerid Ingerid merged commit d9bf4e8 into main Sep 24, 2025
4 checks passed
@Ingerid Ingerid deleted the poetry-to-pdm branch September 24, 2025 14:43
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.

Project management: Convert pyproject.toml from using poetry to using pdm or uv

3 participants