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

feat: Send more workflow info to plugins. Fixes #8573 #8575

Closed
wants to merge 10 commits into from

Conversation

whybeyoung
Copy link
Contributor

Fixes #8573

Signed-off-by: maybaby <ybyang7@iflytek.com>
workflow/executor/agent.go Outdated Show resolved Hide resolved
@alexec alexec changed the title [feat] add more workflow information to plugin feat: Send more workflow info to plugins. Fixes #8573 May 4, 2022
Copy link
Contributor

@alexec alexec left a comment

Choose a reason for hiding this comment

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

This looks great, but does need a test.

@whybeyoung
Copy link
Contributor Author

This looks great, but does need a test.

OK,i will add them

test/e2e/agent_test.go Outdated Show resolved Hide resolved
@alexec alexec added the pinned label Jun 23, 2022
alexec and others added 3 commits August 10, 2022 09:36
Signed-off-by: Alex Collins <alex_collins@intuit.com>
Signed-off-by: Alex Collins <alex_collins@intuit.com>
@alexec alexec closed this Oct 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[plugin] add more workflow data in executor plugin
2 participants