Skip to content

[improve][doc] Update links to the contribute guide#18509

Merged
tisonkun merged 2 commits intoapache:masterfrom
tisonkun:update-links-to-the-contribute-guide
Nov 17, 2022
Merged

[improve][doc] Update links to the contribute guide#18509
tisonkun merged 2 commits intoapache:masterfrom
tisonkun:update-links-to-the-contribute-guide

Conversation

@tisonkun
Copy link
Member

Motivation

This reflects apache/pulsar-site#295.

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

Signed-off-by: tison <wander4096@gmail.com>
@github-actions github-actions bot added the doc Your PR contains doc changes, no matter whether the changes are in markdown or code files. label Nov 16, 2022
@@ -221,134 +219,20 @@ docker push ${image_repo_and_project}/pulsar:$tag

## Setting up your IDE
Copy link
Member Author

Choose a reason for hiding this comment

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

I agree that it helps if we inline this info but it's too long to keep in sync.

Copy link
Contributor

@momo-jun momo-jun left a comment

Choose a reason for hiding this comment

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

LGTM. Only left a couple of minor suggestions.

Co-authored-by: momo-jun <60642177+momo-jun@users.noreply.github.com>
@tisonkun
Copy link
Member Author

@momo-jun Thanks! Comments addressed.

@tisonkun
Copy link
Member Author

Merging...

Review after merge is welcome.

@tisonkun tisonkun merged commit 6f6e32e into apache:master Nov 17, 2022
@tisonkun tisonkun deleted the update-links-to-the-contribute-guide branch November 17, 2022 01:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Your PR contains doc changes, no matter whether the changes are in markdown or code files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants