mobilecoinofficial / forest

Enables a Forest of MobileCoin enabled SignalBots. Beta software, APIs may change!
MIT License
19 stars 12 forks source link

Switch to poetry and pyproject so we're installable as a pip package #148

Closed technillogue closed 2 years ago

technillogue commented 2 years ago

like so: https://github.com/technillogue/rinkbot

template to clone/fork and change: https://github.com/mobilecoinofficial/forestbot-template

itdaniher commented 2 years ago

can we roll mobfriend etc over to poetry too in this same commit? maybe document the migration procedure from pipenv?

technillogue commented 2 years ago

will try to do, slightly tricky because of branch references will stop being valid once it's merged migration procedure is to just write a pyproject.toml following the template in sample_bots/pyproject.toml with any additional dependencies your bot has, poetry lock it, and update the dockerfile following the example in top level Dockerfile (which notably features no longer doing the graal compile job, but makes it harder for us to use unreleased commits)

technillogue commented 2 years ago

not entirely sure it makes sense to update contact, imogen, or echopay since there's work being done on them in unmerged branches?

also needs to use poetry in ci with a superset of deps

technillogue commented 2 years ago

@itdaniher okay, this switches ci, mobfriend, contact, and echopay but not imogen to poetry. what do you think?

this will also resolve #141