Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move optional dependencies from dev_requirements to pyproject.toml #170

Merged
merged 7 commits into from
Oct 7, 2024

Conversation

FreddyFox892
Copy link
Contributor

No description provided.

@hf-kklein
Copy link
Contributor

hey @mj0nez wir ziehen gerade viele unserer python-repos auf dieses setup um und dachten, wir nehmen das hier gleich mit.
der vorteil ist:

  • weniger requirements.txt und dadurch weniger Konflikte zwischen den requirements.txt files
  • insgesamt mehr infos in der pyproject.toml
    der nachteil ist:
  • die indirekten dependencies (auf die z.b. pytest oder pylint selbst aufbauen) sind nicht mehr gepinned. für uns bei Hochfrequenz war das eine bewusste entscheidung es so zu machen. es reduziert die manuellen aufwände aber macht die reproduzierbarkeit ein klein bisschen schlechter

Move optional dependencies from `dev_requirements` to `pyproject.toml`
Copy link
Owner

@mj0nez mj0nez left a comment

Choose a reason for hiding this comment

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

Danke für den Hintergrund und das PR! Ich hab ein paar Anmerkungen hinterlassen. :)

pyproject.toml Outdated Show resolved Hide resolved
.github/workflows/release.yml Outdated Show resolved Hide resolved
pyproject.toml Outdated Show resolved Hide resolved
.github/dependabot.yml Outdated Show resolved Hide resolved
@mj0nez mj0nez enabled auto-merge October 7, 2024 21:09
@mj0nez mj0nez merged commit a0fdc69 into mj0nez:main Oct 7, 2024
18 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.

3 participants