Skip to content
This repository has been archived by the owner on Jan 2, 2025. It is now read-only.

🐛 Simplify __repr__ and test for __repr__ #405

Merged
merged 17 commits into from
Jul 31, 2024
Merged

🐛 Simplify __repr__ and test for __repr__ #405

merged 17 commits into from
Jul 31, 2024

Conversation

Zethson
Copy link
Member

@Zethson Zethson commented Jul 31, 2024 •
edited
Loading

Fixes #404
Fixes #406

  1. Simplify __repr__ by removing code that was not needed anymore.
  2. Add a test for __repr__. Side effect: We now install lamindb[bionty] also in the CI.

Zethson added 10 commits July 31, 2024 13:16
@Zethson Zethson changed the title 🐛 Simplify __repr__ 🐛 Simplify __repr__ and test for __repr__ Jul 31, 2024
@Zethson Zethson merged commit adb13e1 into main Jul 31, 2024
1 check passed
@Zethson Zethson deleted the fix/simplify_repr_ branch July 31, 2024 12:57
@falexwolf falexwolf mentioned this pull request Aug 3, 2024
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add test for __repr__ of Registry Link models in __repr__
1 participant
@Zethson