Skip to content
This repository has been archived by the owner on Dec 10, 2024. It is now read-only.

Library: Use multitrait handle #35

Merged
merged 2 commits into from
Sep 3, 2024
Merged

Conversation

iopapamanoglou
Copy link
Contributor

Library: Use multitrait handle

Description

  • Adapt multi traits: Descriptive; Picker; Linked Pad
  • Move from add_trait to add
  • Add checks to not allow instantiate raw trait

Fixes # (issue)

Checklist

Please read and execute the following:

  • My code follows the coding guidelines of this project
  • My PR title is following the contribution guidelines
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (if applicable)
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works (if applicable)
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules
  • I ran Black to format my code

Code of Conduct

By submitting this issue, you agree to follow our Code of Conduct:

@iopapamanoglou iopapamanoglou merged commit 593fd39 into main Sep 3, 2024
1 check passed
@iopapamanoglou iopapamanoglou deleted the feature/use_multi_traits branch September 3, 2024 15:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant