Skip to content

Conversation

Chaitanyareddy0702
Copy link
Contributor

@Chaitanyareddy0702 Chaitanyareddy0702 commented Sep 9, 2025

Description of your changes

I have:

  • Read and followed KubeVela's contribution process.
  • Update sidebar.js if adding a new page.
  • Run yarn start to ensure the changes has taken effect.

Special notes for your reviewer

@semmet95
Copy link

Doc content is dependent on this PR: kubevela/kubevela#6878

@Chaitanyareddy0702 Chaitanyareddy0702 marked this pull request as ready for review September 25, 2025 04:18
Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

2 issues found across 2 files

Prompt for AI agents (all 2 issues)

Understand the root cause of the following 2 issues and fix them.


<file name="docs/contributor/code-contribute.md">

<violation number="1" location="docs/contributor/code-contribute.md:528">
Process step 2 list omits `kubevela/velaux` despite being in Scope; include it to avoid inconsistent guidance.</violation>

<violation number="2" location="docs/contributor/code-contribute.md:576">
Version placeholder &quot;va.b.c&quot; appears to be a typo; use a conventional format like vX.Y.Z.</violation>
</file>

React with 👍 or 👎 to teach cubic. Mention @cubic-dev-ai to give feedback, ask questions, or re-run the review.

@Chaitanyareddy0702 Chaitanyareddy0702 force-pushed the chore/add-k8s-upgrade-documentation branch from ed76cea to e6b9070 Compare September 30, 2025 06:39
Signed-off-by: Chaitanyareddy0702 <[email protected]>

refactor: minor doc changes

Signed-off-by: Amit Singh <[email protected]>

Chore: Add velaux to the list of KubeVela repositories

Signed-off-by: Chaitanyareddy0702 <[email protected]>
@Chaitanyareddy0702 Chaitanyareddy0702 force-pushed the chore/add-k8s-upgrade-documentation branch from e6b9070 to 08c0565 Compare September 30, 2025 06:40
@anoop2811 anoop2811 merged commit 12f17f9 into kubevela:main Sep 30, 2025
6 checks passed
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.

3 participants