widgetti / solara

A Pure Python, React-style Framework for Scaling Your Jupyter and Web Apps
https://solara.dev
MIT License
1.62k stars 105 forks source link

feature request: add package on conda-forge #105

Closed luk-f-a closed 3 weeks ago

luk-f-a commented 12 months ago

hi devs, great project, I'm a heavy user of ipyvuetify and I really like the concept behind solara. I had a problem when I wanted to test it: there's no package on conda-forge (that I could find, at least).

I know that it's possible to use pip in a conda environment, which I did, but the large number of dependencies became a problem, because pip and conda don't play together very well in dependency resolution. pip attempted to update and reinstall many already-present packages.

Anyway, something to consider for the future. Thanks for your work!

maartenbreddels commented 12 months ago

Ouch, I typed a whole reply, and I lost it :).

Short version:

We currently have a very smooth release process, and in my experience conda-forge adds a lot of friction for releasing, especially with two interdependent packages (solara and solara-assets).

But if we can have a conda-forge recipe that releases both solara and solara-assets in 1 PR, and if we can open that PR from https://github.com/widgetti/solara/blob/master/.github/workflows/release.yaml I'm good with it. Then in principle releasing is just clicking a merge button on conda-forge.

If someone wants to set that up, I'm in favor.

giswqs commented 11 months ago

I have submitted a PR to add solara and solara-assets to conda-forge (https://github.com/conda-forge/staged-recipes/pull/22935). Once the recipes have been accepted, I can add more maintainers. We can set up a bot automerge, which can automatically create PR based on PyPI releases. We only need to manually update the conda-forge recipe if the dependency list of the PyPI package has changed.

giswqs commented 11 months ago

@maartenbreddels Can you merge the PR (https://github.com/conda-forge/ipyvuetify-feedstock/pull/39) to update ipyvuetify to v1.8.10? Solara requires ipyvuetify >=1.8.10.

iisakkirotko commented 3 weeks ago

Closing this as done by https://github.com/conda-forge/staged-recipes/pull/23413. The recipe can be found at https://github.com/conda-forge/solara-feedstock.