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

Add documentation to release Helm Chart #13385

Merged
merged 1 commit into from
Jan 15, 2023
Merged

Add documentation to release Helm Chart #13385

merged 1 commit into from
Jan 15, 2023

Conversation

kezhenxu94
Copy link
Member

Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.

(or)

This pull request is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

(or)

If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

@@ -16,7 +16,7 @@
#

apiVersion: v2
name: dolphinscheduler
name: dolphinscheduler-helm
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For release managers, if you plan to release Helm Chart for old versions, you need to modify this

SbloodyS
SbloodyS previously approved these changes Jan 12, 2023
Copy link
Member

@SbloodyS SbloodyS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@EricGao888 EricGao888 added the improvement make more easy to user or prompt friendly label Jan 13, 2023
@EricGao888 EricGao888 added this to the 3.2.0 milestone Jan 13, 2023
@zhongjiajie
Copy link
Member

FYI, CI failed

@sonarcloud
Copy link

sonarcloud bot commented Jan 13, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@kezhenxu94 kezhenxu94 added the 3.2.0 for 3.2.0 version label Jan 13, 2023
Copy link
Contributor

@caishunfeng caishunfeng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@caishunfeng caishunfeng merged commit a843e0b into dev Jan 15, 2023
@kezhenxu94 kezhenxu94 deleted the release/helm branch January 15, 2023 02:10
zhuangchong added a commit that referenced this pull request Apr 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.2.0 for 3.2.0 version document improvement make more easy to user or prompt friendly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants