Skip to content
New issue

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

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

Already on GitHub? # to your account

[CI] update CI to keep same with github action template #6

Merged
merged 8 commits into from
Dec 21, 2023
Merged

Conversation

SamYuan1990
Copy link
Contributor

@SamYuan1990 SamYuan1990 commented Dec 21, 2023

test security group for #5

update CI with ci.yml as before and after PR level check, make it same with our example in readme document to keep user's input been tested with our CI.
decouple a PR image build and publish process with before and after PR check.
fix lint issue
fix action input define issue
fix action output define issue.

Signed-off-by: Sam Yuan <yy19902439@126.com>
Signed-off-by: Sam Yuan <yy19902439@126.com>
Signed-off-by: Sam Yuan <yy19902439@126.com>
Signed-off-by: Sam Yuan <yy19902439@126.com>
Signed-off-by: Sam Yuan <yy19902439@126.com>
Signed-off-by: Sam Yuan <yy19902439@126.com>
Signed-off-by: Sam Yuan <yy19902439@126.com>
@SamYuan1990 SamYuan1990 requested a review from a team as a code owner December 21, 2023 15:20
Signed-off-by: Sam Yuan <yy19902439@126.com>
@SamYuan1990 SamYuan1990 requested a review from rootfs December 21, 2023 15:26
@SamYuan1990 SamYuan1990 changed the title Update ci [CI] update CI to keep same with github action template Dec 21, 2023
@rootfs rootfs merged commit 44d7ac1 into main Dec 21, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants