Skip to content

website: Workspace contributor docs streamline - #4476

Open
christian-heusel wants to merge 2 commits into
kubeflow:masterfrom
christian-heusel:chore/workspaces-docs-changes
Open

website: Workspace contributor docs streamline#4476
christian-heusel wants to merge 2 commits into
kubeflow:masterfrom
christian-heusel:chore/workspaces-docs-changes

Conversation

@christian-heusel

Copy link
Copy Markdown
Member

Description of Changes

  • chore: Add workspaces as accepted PR prefix
  • docs: Unify the current contributor guides

Related Issues

None

Checklist

Signed-off-by: Christian Heusel <christian@heusel.eu>
@google-oss-prow

Copy link
Copy Markdown

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@google-oss-prow

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign franciscojavierarceo for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Signed-off-by: Christian Heusel <christian@heusel.eu>
@christian-heusel
christian-heusel force-pushed the chore/workspaces-docs-changes branch from e38bf58 to 7892518 Compare August 28, 2026 21:28
@christian-heusel
christian-heusel marked this pull request as ready for review August 28, 2026 21:29
@christian-heusel

Copy link
Copy Markdown
Member Author

cc @HarshPopat23 for a round of review

@HarshPopat23 HarshPopat23 left a comment

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.

Thanks for streamlining the contributor documentation-the consolidation makes sense, and the Task, Feature, and Epic links are preserved.

I noticed two items before LGTM:

  1. planning-process.md is deleted, but I could not find an alias or redirect for its existing public URL. Could we redirect /docs/components/workspaces/contributor-guides/planning-process/ to the new contribute/#planning-process section so existing links and bookmarks do not return 404?

  2. The title validator now accepts workspaces, and COMMENT_BODY lists it, but the terminal echo "Component value must be..." message still omits workspaces. That message should be updated for consistency.

Apart from these points, the overall direction looks very good.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants