Skip to content

Open to more features? #4

Closed
Closed

Description

@stefanmaric

Hi @pydsigner 👋

First of all, thanks for your contributions to the community through this package. ❤️

I needed to process some CSS on the fly in a Flask project and this module has been very helpful.

Since you have stated already that this project is "Opinionated" and "only a subset of functionality is exposed", I'm assuming you don't want to add additional features.

One feature I was missing was support for bundling, so I had to fork and publish a diff version: https://test.pypi.org/project/lightningcss-bindings/

I'm planning on expanding the bindings to be a more generic solution, but wanted to first check with you in case you prefer my contributions to be integrated in your project.

I've opened a pair of Pull Requests for chore tasks to give back to you:

#2
#3

Feel free to close them if you don't like it, I won't be mad!

Let me know your thoughts about collaborating on new features for python-lightningcss! I will wait a while before formalizing my fork and publishing to PyPi.

Thanks again!

Activity

  1. pydsigner commented on Jun 30, 2024

    @pydsigner
    Owner

    Hi @stefanmaric, thanks for reaching out— I'm really glad to hear that you've found these bindings useful already! I did describe the bindings as opinionated, but that's more of a present tense thing, since there's just the one entry point and minimal flexibility. I would like to expose more functionality. #1 has been the big item in my perspective, since it would allow me to get rid of a bunch of arcane code in another project, but I'm open to items like bundling being exposed also.

  2. pydsigner commented on Jun 30, 2024

    @pydsigner
    Owner

    I did take a look at your fork and noticed that you had dropped support for older Pythons. I would like to get further with the project before dropping any Python that hasn't at least reached EOL. (Please disregard if that was just to make builds run faster for testing...)

  3. stefanmaric commented on Jun 30, 2024

    @stefanmaric
    ContributorAuthor

    thanks for reaching out

    Thanks for getting back!

    (Please disregard if that was just to make builds run faster for testing...)

    I just talked about this in #3, it was definitely for testing 😄

    but I'm open to items like bundling being exposed also

    @pydsigner so you have the time and energy to manage my contributions and publish new releases? - if so, I can prepare a PR to integrate the bundling function (which I'm actually using already via TestPyPi for the project I needed it for) and then work on more features (I was actually planning to write some integration tests soon to make the dev cycle shorter, to not have to publish to TestPyPi to verify the changes).

  4. pydsigner commented on Jun 30, 2024

    @pydsigner
    Owner

    Yes I should be able to be fairly responsive on that— I've automated the release process enough that the pain of releases themselves is quite limited, and I have system tests in https://github.com/pydsigner/anchovy I can use for regression detection though tests are currently lacking here (and getting some integration tests here will improve that further).

  5. mafr commented on Dec 13, 2025

    @mafr
    Contributor

    Friendly ping :)

    Exposing bundling functionality would be what makes this interesting for me. I'm hoping to use this via the webassets module one day and get rid of SASS.

  6. pydsigner commented on Dec 14, 2025

    @pydsigner
    Owner

    @mafr I never got a PR from @stefanmaric but the bundle CSS code he wrote is here: master...stefanmaric:python-lightningcss:upgrades. It would need some adjustments to be mergeable but shouldn't be too hard to cherrypick into a PR if you're willing to take that on. I would certainly be willing to review and merge the change + make a release!

  7. pydsigner commented on Dec 14, 2025

    @pydsigner
    Owner

    I would like to move to some sort of more modular structure to reduce how much code path duplication occurs but I'm aware that that's a longer-term goal and potentially much more work.

  8. mafr commented on Dec 14, 2025

    @mafr
    Contributor

    I have no Rust skills whatsoever, but I'll see if I can cherry-pick that commit and see if it works. Thank you!

  9. pydsigner commented on Dec 16, 2025

    @pydsigner
    Owner

    Resolved in #8. I will open a new issue to track creation of tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions