New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

docs: polish the project organization doc #656

Merged

d4x1 merged 1 commit into apache:main from Startrekzky:doc-louis

Oct 11, 2023

Contributor

Startrekzky commented Oct 10, 2023

⚠️ Pre Checklist

Please complete ALL items in this checklist, and remove before submitting

I have npm run build and npm run serve locally before submitting this PR
I have read through the Contributing Documentation

Summary

Restructure the doc
Simplify the wording and remove redundant parts
Clarify the uses for team and project
Update the picture for use-case-2
Change the position of this doc in the directory
Standardize to second and third person
Improve the format
and more.

Does this close any open issues?

The refinement of #637

Screenshots

Other Information

Any other information that is important to this PR.


          docs: polish the project organization doc

c90923e

KucherenkoSerhiy approved these changes

View reviewed changes

Contributor

KucherenkoSerhiy left a comment

The guide looks much cleaner, concise, and to the point with these changes! Great job!

d4x1 approved these changes

View reviewed changes

Contributor

d4x1 left a comment

LGTM

d4x1 merged commit 0f3ce7f into apache:main

1 check passed

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

Labels

None yet