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

Core: More convenient data-traits #67

Merged
merged 1 commit into from
Sep 23, 2024

Conversation

mawildoer
Copy link
Contributor

@mawildoer mawildoer commented Sep 19, 2024

Description

Add more convenient traits for simple applications.

This convenience comes from allowing decless (declaration-less) implementations, which actually have an implicit declaration.

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:

@mawildoer mawildoer force-pushed the mawildoer/reduced-trait-verbosity branch from c8e431d to 22d702d Compare September 19, 2024 14:03
Additionally, simplify the trait impl implementation and extend trait testing
@mawildoer mawildoer force-pushed the mawildoer/reduced-trait-verbosity branch from 22d702d to f38e60e Compare September 19, 2024 14:51
@mawildoer mawildoer changed the title WIP: more conveneient data-traits Core: More convenient data-traits Sep 19, 2024
@mawildoer mawildoer marked this pull request as ready for review September 19, 2024 14:58
Copy link
Contributor

@iopapamanoglou iopapamanoglou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very clean!

@mawildoer mawildoer merged commit 68372ff into main Sep 23, 2024
2 checks passed
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.

2 participants