Skip to content

fix some doc error#3551

Merged
zhoujinsong merged 1 commit into
apache:masterfrom
Aireed:fix_amoro_doc
May 14, 2025
Merged

fix some doc error#3551
zhoujinsong merged 1 commit into
apache:masterfrom
Aireed:fix_amoro_doc

Conversation

@Aireed
Copy link
Copy Markdown
Contributor

@Aireed Aireed commented May 8, 2025

Why are the changes needed?

fix some doc error, keep same with amoro-site repo

Brief change log

How was this patch tested?

  • Add some test cases that check the changes thoroughly including negative and positive cases if possible

  • Add screenshots for manual tests if appropriate

  • Run test locally before making a pull request

Documentation

  • Does this pull request introduce a new feature? (yes / no)
  • If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@github-actions github-actions Bot added the type:docs Improvements or additions to documentation label May 8, 2025
```

Alternatively, you can find the latest charts directly from the Github source code.
You can find the latest charts directly from the Github source code.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

refer to a link?

$ helm repo add amoro https://netease.github.io/amoro/charts
$ helm search repo amoro
NAME CHART VERSION APP VERSION DESCRIPTION
amoro/amoro 0.1.0 0.7.0 A Helm chart for Amoro
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

APP VERSION need to update: 0.8.0

Copy link
Copy Markdown
Contributor

@zhoujinsong zhoujinsong left a comment

Choose a reason for hiding this comment

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

LGTM.

@zhoujinsong zhoujinsong merged commit c0c11fe into apache:master May 14, 2025
1 check passed
@Aireed Aireed deleted the fix_amoro_doc branch May 15, 2025 07:17
@Jzjsnow Jzjsnow mentioned this pull request Aug 11, 2025
52 tasks
Jzjsnow pushed a commit to Jzjsnow/amoro that referenced this pull request Aug 20, 2025
fix#keep same with amoro-site repo

(cherry picked from commit c0c11fe)
zhoujinsong pushed a commit that referenced this pull request Aug 25, 2025
fix#keep same with amoro-site repo

(cherry picked from commit c0c11fe)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:docs Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants