Skip to content

feat: support dependencies from pyproject.toml#764

Merged
amol- merged 2 commits into
mainfrom
pyproject
May 12, 2026
Merged

feat: support dependencies from pyproject.toml#764
amol- merged 2 commits into
mainfrom
pyproject

Conversation

@amol-
Copy link
Copy Markdown
Collaborator

@amol- amol- commented Apr 29, 2026

Intent

Allow to pass --requirements-file pyproject.toml to rsconnect deploy so that it is not necessary to create a redundant requirements.txt

Type of Change

  • Bug Fix
  • New Feature
  • Breaking Change

Approach

Mimic #749

Automated Tests

added some dedicated tests

Directions for Reviewers

  1. delete any requirements.txt
  2. deploy with --requirements-file pyproject.toml

Checklist

  • I have updated CHANGELOG.md to cover notable changes.
  • I have updated all related GitHub issues to reflect their current state.
  • I have run the rsconnect-python-tests-at-night workflow in Connect against this feature branch.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 29, 2026

PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-05-12 14:18 UTC

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 29, 2026

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
5642 4347 77% 0% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
rsconnect/main.py 69% 🟢
rsconnect/subprocesses/inspect_environment.py 88% 🟢
TOTAL 79% 🟢

updated for commit: edb9b3e by action🐍

Copy link
Copy Markdown
Collaborator

@tdstein tdstein left a comment

Choose a reason for hiding this comment

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

Are you considering [project.optional-dependencies] support as well?

@amol-
Copy link
Copy Markdown
Collaborator Author

amol- commented Apr 30, 2026

Are you considering [project.optional-dependencies] support as well?

It's something we should design for, but I considered it out of scope as we don't seem to have any support for optional dependencies in general at the moment (independently from pyproject)

@amol- amol- marked this pull request as ready for review May 12, 2026 10:26
@amol- amol- requested a review from tdstein May 12, 2026 10:26
@amol- amol- merged commit f6cd955 into main May 12, 2026
23 checks passed
@amol- amol- deleted the pyproject branch May 12, 2026 14:18
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