Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Components should not be blueprint types. #139

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Conversation

iago-lito
Copy link
Collaborator

@iago-lito iago-lito commented Mar 15, 2024

Address this design mistake: blueprints and components should be two separate types hierarchies.

Using singleton component instances as ergonomical handles should improve #131.

Take this opportunity to better unify brought blueprint, either implied, embedded or unbrought depending on blueprint value.

This will surface as breaking API changes, but I expect them to remain minor. One major positive change I expect is the simplification of all code within src/components/: this should not become so complex than no one can hack in.

@iago-lito iago-lito self-assigned this Mar 15, 2024
@iago-lito iago-lito deleted the branch main March 19, 2024 08:49
@iago-lito iago-lito closed this Mar 19, 2024
@iago-lito iago-lito reopened this Mar 19, 2024
@iago-lito iago-lito changed the base branch from api to dev March 19, 2024 09:21
@iago-lito iago-lito marked this pull request as draft March 19, 2024 10:07
@iago-lito iago-lito force-pushed the dev branch 7 times, most recently from 1684652 to 5d1c9e5 Compare March 25, 2024 16:01
@iago-lito iago-lito force-pushed the blueprints branch 12 times, most recently from 5a159d5 to 31c7d43 Compare July 30, 2024 15:17
@iago-lito iago-lito force-pushed the blueprints branch 2 times, most recently from cf1f51d to ae1e0ee Compare September 30, 2024 11:39
@iago-lito iago-lito force-pushed the blueprints branch 5 times, most recently from e4711f3 to 1609a56 Compare October 24, 2024 09:30
@iago-lito iago-lito force-pushed the blueprints branch 2 times, most recently from 63439f8 to 598f7e2 Compare November 8, 2024 15:04
@iago-lito iago-lito force-pushed the blueprints branch 4 times, most recently from b362b6b to 053855d Compare December 16, 2024 16:29
@iago-lito iago-lito force-pushed the blueprints branch 3 times, most recently from c8f31df to 11e6cb9 Compare December 20, 2024 08:39
@iago-lito iago-lito marked this pull request as ready for review December 20, 2024 13:35
@iago-lito iago-lito force-pushed the blueprints branch 5 times, most recently from e59db3e to 801f34f Compare December 20, 2024 20:57
@iago-lito iago-lito changed the base branch from dev to main December 20, 2024 20:57
iago-lito added a commit that referenced this pull request Dec 20, 2024
This deep refactoring of the Framework addresses and fixes #139.
It fixes the semantics of the components library
(*e.g.* `OmegaFromRawEdges` and `OmegaFromAllometry`
are *not* two separate components)
while making it more flexible and future-proof.

Take this opportunity to tick the numerous additional Framework `TODO`s,
contributing to polishing the whole library API experience.

Summarize changes in a fresh `CHANGELOG.md`, suggesting to issue v0.2.1.
This deep refactoring of the Framework addresses and fixes #139.
It fixes the semantics of the components library
(*e.g.* `OmegaFromRawEdges` and `OmegaFromAllometry`
are *not* two separate components)
while making it more flexible and future-proof.

Take this opportunity to tick the numerous additional Framework `TODO`s,
contributing to polishing the whole library API experience.

Summarize changes in a fresh `CHANGELOG.md`, suggesting to issue v0.2.1.
@iago-lito
Copy link
Collaborator Author

This is finally ready for review 🎉 As expected, the refactoring surfaced as minor breaking API changes, so we'd need to bump our version number to merge this. See the CHANGELOG for details.
Try it out! Criticize! Find bugs :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant