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

Context used for Atom interface should be custom and supply the necessary integrations for the other parameters of the Process method #80

Open
benjivesterby opened this issue Oct 29, 2021 · 0 comments
Assignees
Labels
enhancement New feature or request Research

Comments

@benjivesterby
Copy link
Member

This change would reduce the number of expected parameters for the Process method to only context.Context which should be supplied by the Atomizer engine. Since context.Context is an interface the Atomizer engine should implement the interface internally and supply an Atomizer Context which when used in conjunction with a type assertion can give access to the Atomizer communication API as well as other necessary engine APIs.

@benjivesterby benjivesterby added enhancement New feature or request Research labels Oct 29, 2021
@benjivesterby benjivesterby self-assigned this Oct 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Research
Projects
None yet
Development

No branches or pull requests

1 participant