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

flowey: streamline artifact node declarations #377

Open
daprilik opened this issue Nov 21, 2024 · 0 comments
Open

flowey: streamline artifact node declarations #377

daprilik opened this issue Nov 21, 2024 · 0 comments
Labels
flowey Improvements to the flowey build infrastructure

Comments

@daprilik
Copy link
Contributor

daprilik commented Nov 21, 2024

@daprilik, artifact_hypestv in particular seems like boilerplate that could be shared across all the host-side tools. Am I wrong?

Yes, I think there's certainly an avenue to consolidate that sort of common "simple rust bin" artifact logic. I think there are a few approaches on how to achieve that, though the one that I lean towards is doing something similar to SimpleFlowNode, where you can implement a special ArtifactFlowNode trait which has overrides for various artifact-specific things (e.g: bin name, etc..), and have it then impl the FlowNode for you.

Originally posted by @daprilik in #375 (comment)

@daprilik daprilik added the flowey Improvements to the flowey build infrastructure label Nov 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
flowey Improvements to the flowey build infrastructure
Projects
None yet
Development

No branches or pull requests

1 participant