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

chore: release helm chart when the tag starts with helm #726

Merged
merged 3 commits into from
Nov 27, 2023
Merged

Conversation

wangxye
Copy link
Contributor

@wangxye wangxye commented Nov 24, 2023

change:

  1. release when the tag starts with helm;
  2. docker build for helm chart;

Copy link

codecov bot commented Nov 24, 2023

Codecov Report

Merging #726 (c6b4d12) into main (e7f310d) will decrease coverage by 0.03%.
Report is 6 commits behind head on main.
The diff coverage is n/a.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #726      +/-   ##
============================================
- Coverage     57.47%   57.45%   -0.03%     
+ Complexity     1224     1222       -2     
============================================
  Files           131      131              
  Lines          8029     8031       +2     
  Branches        736      736              
============================================
- Hits           4615     4614       -1     
- Misses         3019     3020       +1     
- Partials        395      397       +2     
Components Coverage Δ
RocketMQ Broker ∅ <ø> (∅)
RocketMQ Common 5.80% <ø> (ø)
RocketMQ Controller 54.88% <ø> (ø)
RocketMQ Proxy 42.75% <ø> (ø)
RocketMQ Store 76.96% <ø> (-0.13%) ⬇️
RocketMQ Stream ∅ <ø> (∅)
RocketMQ Metadata 35.29% <ø> (ø)

see 2 files with indirect coverage changes

@lizhanhui
Copy link
Contributor

Keep original release workflow as it is and add a new workflow to release helm charts

@lizhanhui lizhanhui merged commit aa5f1ef into main Nov 27, 2023
15 of 16 checks passed
@lizhanhui lizhanhui deleted the chart_relase branch November 27, 2023 09:24
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.

2 participants