utilities icon indicating copy to clipboard operation
utilities copied to clipboard

Apply func non roundtrippable seq

Open ap-- opened this issue 1 year ago • 1 comments

Before submitting
  • [x] Was this discussed/agreed via a Github issue? (no need for typos and docs improvements)
  • [x] Did you read the contributor guideline, Pull Request section?
  • Did you make sure to update the docs?
  • [x] Did all existing and newly added tests pass locally?

What does this PR do?

Fixes #249

Note: I'm not sure if the added overhead in the slow path of apply_to_collection due to the shallow sequence copy and the elementwise comparison is undesirable to support this use case. I could reimplement this to cache the result of can_roundtrip_sequence based on the class.

PR review

Anyone in the community is free to review the PR once the tests have passed. If we didn't discuss your PR in GitHub issues there's a high chance it will not be merged.

Cheers, Andreas 😃


📚 Documentation preview 📚: https://lit-utilities--250.org.readthedocs.build/en/250/

ap-- avatar Mar 29 '24 17:03 ap--

Welcome to Codecov :tada:

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered :open_umbrella:

codecov[bot] avatar Mar 29 '24 22:03 codecov[bot]

Closing in favour of upcoming PR implementing an exclude option for user provided types.

ap-- avatar Jun 27 '24 09:06 ap--

Closing in favour of upcoming PR implementing an exclude option for user provided types.

Thank you. Do you mind also having a look at #279? :pray:

Borda avatar Jul 13 '24 07:07 Borda