Skip to content

fix up #196

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

Closed
wants to merge 5 commits into from
Closed

Conversation

SamYuan1990
Copy link
Collaborator

@SamYuan1990 SamYuan1990 commented Sep 17, 2022

resolve #193

change log:

  • add commit push condition for main branch.
  • move test coverage for default unit test.(to avoid test coverage based on specific build tag as bcc)
  • bug fix for test coverage file missing.

Signed-off-by: Sam Yuan [email protected]

Signed-off-by: Sam Yuan <[email protected]>
@SamYuan1990 SamYuan1990 marked this pull request as draft September 17, 2022 11:44
Signed-off-by: Sam Yuan <[email protected]>
@rootfs
Copy link
Contributor

rootfs commented Sep 17, 2022

@SamYuan1990 this is one step forward. There is a permission to given to the bot, let me know what you need and we start from there.

Signed-off-by: Sam Yuan <[email protected]>
@SamYuan1990
Copy link
Collaborator Author

@SamYuan1990 this is one step forward. There is a permission to given to the bot, let me know what you need and we start from there.

well, I found ad-m/github-push-action#96 this one as open issue. still in attempt and investigation.

Signed-off-by: Sam Yuan <[email protected]>
Signed-off-by: Sam Yuan <[email protected]>
@SamYuan1990
Copy link
Collaborator Author

will close this pr for now, moving to SamYuan1990#2 for testing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

cover: open coverage.out: no such file or directory
2 participants