Skip to content

Conversation

@steinliber
Copy link

@steinliber steinliber commented Jul 20, 2022

Signed-off-by: Meng JiaFeng jiafeng.meng@merico.dev

Pre-Checklist

Note: please complete ALL items in the following checklist.

  • I have read through the CONTRIBUTING.md documentation.
  • My code has the necessary comments and documentation (if needed).
  • I have added relevant tests

Description

  • refactor all githubaction plugins, including golang, nodejs, and python
  • all plugins have been tested

Related Issues

Subtask of #850

New Behavior (screenshots if needed)

Current golang githubaction create method
image

@steinliber steinliber requested a review from a team as a code owner July 20, 2022 01:09
@steinliber steinliber added the refactor refactor label Jul 20, 2022
@iyear
Copy link
Member

iyear commented Jul 20, 2022

Very nice refactoring. Very elegant! 🚀

Signed-off-by: Meng JiaFeng <jiafeng.meng@merico.dev>
@steinliber steinliber force-pushed the feat-refactor-github-plugin-installer branch from adb1c86 to b55a4bd Compare July 20, 2022 07:01
@steinliber steinliber requested a review from iyear July 20, 2022 07:01
@IronCore864
Copy link
Member

Great work. Merging now.

@IronCore864 IronCore864 merged commit 1df29fd into devstream-io:main Jul 20, 2022
@IronCore864
Copy link
Member

@steinliber please make sure the latest e2e-test running on the main branch after merging is successful.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor refactor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants