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

Add support for specifying custom dependency groups in Poetry #746

Merged
merged 1 commit into from
Jan 8, 2023

Conversation

mLupine
Copy link
Contributor

@mLupine mLupine commented Nov 30, 2022

Poetry 1.2+ allows to create custom dependency groups in pyproject.toml. However, the export plugin (unlike the install command) does not include those dependencies by default, even if they're not marked as optional, resulting in them being missing from the Serverless deployment package. This PR introduces three new configuration parameters that directly wrap Poetry's --with, --without and --only parameters and allow to include dependencies from those missing groups in the exported requirements.txt file.

@pgrzesik
Copy link
Contributor

Hey @mLupine - thanks a lot for the PR. I'll try to review it by the end of the week but I don't promise anything as I've been way too busy lately.

Copy link
Contributor

@pgrzesik pgrzesik left a comment

Choose a reason for hiding this comment

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

Hey @mLupine, thanks a lot for your patience, unfortunatelly too many things happened at the end of the year and I'm just now catching up on some of the things here. The proposed Pr looks amazing, thanks a lot 🙇

@pgrzesik pgrzesik merged commit 1436c17 into serverless:master Jan 8, 2023
@norayr-im
Copy link

Any updates?

mLupine added a commit to mLupine/serverless-python-requirements that referenced this pull request Nov 3, 2023
* master:
  Release v6.0.1 (serverless#793)
  ci: Temporarily disable test run on integrate (serverless#800)
  ci: Temporarily minimize testing matrix (serverless#799)
  ci: Fix test skips (serverless#797)
  ci: Temp skip of cache-related tests (serverless#796)
  ci: Remove node12 from testing matrix (serverless#795)
  fix: Not crash when runtime is not `python` (serverless#773)
  fix: Remove outdated Pipenv requirements flag (serverless#780)
  fix: Fix integration test matrix configuration (serverless#755)
  fix: Add legacy `pipenv` backward compatability (serverless#742)
  Add support for specifying custom dependency groups in Poetry (serverless#746)
  docs: Add contributing and code of conduct

# Conflicts:
#	test.js
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants