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

ci: fix license script #602

Merged
merged 1 commit into from
May 24, 2022
Merged

ci: fix license script #602

merged 1 commit into from
May 24, 2022

Conversation

seeflood
Copy link
Member

@seeflood seeflood commented May 24, 2022

Signed-off-by: seeflood [email protected]

What this PR does:
Fix make license.
Currently our license script can't work due to the wrong path

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?:


Signed-off-by: seeflood <[email protected]>
@mosn-community-bot mosn-community-bot bot added bug Something isn't working cla:yes size/M labels May 24, 2022
@codecov
Copy link

codecov bot commented May 24, 2022

Codecov Report

Merging #602 (ff2f418) into main (4a8783d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #602   +/-   ##
=======================================
  Coverage   60.70%   60.70%           
=======================================
  Files         120      120           
  Lines        6382     6382           
=======================================
  Hits         3874     3874           
  Misses       2139     2139           
  Partials      369      369           
Impacted Files Coverage Δ
pkg/wasm/abi.go 8.69% <ø> (ø)
sdk/go-sdk/client/secret.go 100.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4a8783d...ff2f418. Read the comment docs.

@seeflood seeflood merged commit 5bada7f into mosn:main May 24, 2022
@seeflood seeflood deleted the fix_license branch May 24, 2022 03:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working cla:yes size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants