Skip to content

Doc: merge docs for CBDB compilation on Linux systems#224

Merged
my-ship-it merged 1 commit intoapache:mainfrom
TomShawn:merge-linux-compile-docs
Oct 30, 2023
Merged

Doc: merge docs for CBDB compilation on Linux systems#224
my-ship-it merged 1 commit intoapache:mainfrom
TomShawn:merge-linux-compile-docs

Conversation

@TomShawn
Copy link
Copy Markdown
Contributor

@TomShawn TomShawn commented Sep 27, 2023

The docs for compiling CBDB on Linux systems are distributed in 2 files: readmes/README.Linux.md and readmes/README.md. This is not user-friendly enough. This PR merges the content into one document for a better reading experience.


Change logs

Describe your change clearly, including what problem is being solved or what feature is being added.

If it has some breaking backward or forward compatibility, please clary.

Why are the changes needed?

Describe why the changes are necessary.

Does this PR introduce any user-facing change?

If yes, please clarify the previous behavior and the change this PR proposes.

How was this patch tested?

Please detail how the changes were tested, including manual tests and any relevant unit or integration tests.

Contributor's Checklist

Here are some reminders and checklists before/when submitting your pull request, please check them:

  • Make sure your Pull Request has a clear title and commit message. You can take git-commit template as a reference.
  • Sign the Contributor License Agreement as prompted for your first-time contribution(One-time setup).
  • Learn the coding contribution guide, including our code conventions, workflow and more.
  • List your communication in the GitHub Issues or Discussions (if has or needed).
  • Document changes.
  • Add tests for the change
  • Pass make installcheck
  • Pass make -C src/test installcheck-cbdb-parallel
  • Feel free to @cloudberrydb/dev team for review and approval when your PR is ready🥳

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

Hiiii, @TomShawn welcome!🎊 Thanks for taking the effort to make our project better! 🙌 Keep making such awesome contributions!

@TomShawn
Copy link
Copy Markdown
Contributor Author

@tuhaihe @Zhangbaowen-Hashdata PTAL, thanks!

Copy link
Copy Markdown
Member

@tuhaihe tuhaihe left a comment

Choose a reason for hiding this comment

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

Great work!

  1. Could you update your commit title by following our commit conventions?

  2. Do we need to update the readmes/README.md file?

Comment thread readmes/README.Linux.md
@TomShawn TomShawn force-pushed the merge-linux-compile-docs branch 2 times, most recently from e7ecb91 to 1a2c244 Compare September 28, 2023 02:57
@TomShawn TomShawn force-pushed the merge-linux-compile-docs branch from 821850a to 9f5cc3e Compare October 25, 2023 10:46
@tuhaihe tuhaihe changed the title docs: merge docs for CBDB compilation on Linux systems Doc: merge docs for CBDB compilation on Linux systems Oct 26, 2023
@TomShawn TomShawn force-pushed the merge-linux-compile-docs branch from 9f5cc3e to 02fb73a Compare October 26, 2023 03:41
@tuhaihe tuhaihe requested a review from Ray-Eldath October 26, 2023 03:48
Comment thread readmes/README.Linux.md Outdated
@TomShawn TomShawn force-pushed the merge-linux-compile-docs branch 2 times, most recently from 141dba7 to 8df0831 Compare October 26, 2023 05:59
tuhaihe
tuhaihe previously approved these changes Oct 26, 2023
Copy link
Copy Markdown
Member

@tuhaihe tuhaihe left a comment

Choose a reason for hiding this comment

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

LGTM.

Comment thread readmes/README.Linux.md Outdated
@TomShawn TomShawn force-pushed the merge-linux-compile-docs branch from b96da17 to 0e2d4d7 Compare October 27, 2023 10:15
The docs for compiling CBDB on Linux systems are distributed in 2 files:
readmes/README.Linux.md and readmes/README.md. This is not user-friendly
enough.

This PR merges the content into one document for a better reading
experience.
@TomShawn TomShawn force-pushed the merge-linux-compile-docs branch from 0e2d4d7 to 6514891 Compare October 27, 2023 10:18
Copy link
Copy Markdown
Contributor

@my-ship-it my-ship-it left a comment

Choose a reason for hiding this comment

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

LGTM

@my-ship-it my-ship-it merged commit 5a1fff5 into apache:main Oct 30, 2023
@TomShawn TomShawn deleted the merge-linux-compile-docs branch October 30, 2023 05:49
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.

4 participants