Skip to content

Modularize package install tools and support poetry#816

Closed
bsdz wants to merge 25 commits intoheroku:mainfrom
bsdz:master
Closed

Modularize package install tools and support poetry#816
bsdz wants to merge 25 commits intoheroku:mainfrom
bsdz:master

Conversation

@bsdz
Copy link
Copy Markdown

@bsdz bsdz commented Apr 9, 2019

This PR adds support for poetry and modularizes the package system with the potential ability to include other package managers in future.

It's not 100% perfect yet. Eg, could extract runtime.txt value from pyproject.toml file too and should perhaps clean the build cache if a different package manager introduced.

@bsdz
Copy link
Copy Markdown
Author

bsdz commented Apr 17, 2019

Looks like one of the automated checks tests whether a ruby environment works against a python buildpack. This is the only one that fails.

@bsdz bsdz mentioned this pull request Jun 20, 2019
@CaseyFaist
Copy link
Copy Markdown
Contributor

Hi @bsdz!

Thanks for the pr! ❤️ 🌮 This is a lot of changes at once, and I'm curious if we could focus in on just the method checks for pipenv and pip. There are several competing poetry support PRs open and some changes to the build processes going on, but I'm interested in the logic added to utils here.

I have to trigger tests from my account anyway, so if you have no objections, I can cherry-pick the relevant commits and move forward with this detection logic 👍

@CaseyFaist CaseyFaist mentioned this pull request Oct 3, 2019
@bsdz
Copy link
Copy Markdown
Author

bsdz commented Oct 3, 2019

I have to trigger tests from my account anyway, so if you have no objections, I can cherry-pick the relevant commits and move forward with this detection logic

Please do, you can take what you need. I look forward to seeing 1st class support for poetry (and perhaps other dependency tools too)

:-)

more verbose logging
@bsdz bsdz requested a review from a team as a code owner October 25, 2019 05:54
@edmorley edmorley changed the base branch from master to main August 3, 2020 13:28
@bsdz
Copy link
Copy Markdown
Author

bsdz commented Jun 27, 2021

I recently switch from a buildpack based on this PR to using a helper buildpack instead:

https://github.com/moneymeets/python-poetry-buildpack.git

@bsdz bsdz closed this Jun 27, 2021
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.

2 participants