Skip to content
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.

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

README: simplify the README to refer to the documentation #1380

Merged
merged 1 commit into from
Sep 11, 2023

Conversation

mtardy
Copy link
Member

@mtardy mtardy commented Aug 21, 2023

A lot of information was moved to the documentation so we ended up with duplicate of stuff between the doc website and the README. This should simplify the situation with a smaller README redirecting to the more complete documentation.

Preview the README

@mtardy mtardy added the area/documentation Improvements or additions to documentation label Aug 21, 2023
@mtardy mtardy requested a review from a team as a code owner August 21, 2023 18:09
@mtardy mtardy requested a review from olsajiri August 21, 2023 18:09
@mtardy mtardy force-pushed the pr/mtardy/readme-simplification branch 3 times, most recently from 15f10ef to e419103 Compare August 22, 2023 07:45
A lot of information was moved to the documentation so we ended up with
duplicate of stuff between the doc website and the README. This should
simplify the situation with a smaller README redirecting to the more
complete documentation.

Signed-off-by: Mahe Tardy <mahe.tardy@gmail.com>
@mtardy mtardy force-pushed the pr/mtardy/readme-simplification branch from e419103 to b72a6dc Compare August 22, 2023 07:50
@mtardy
Copy link
Member Author

mtardy commented Aug 25, 2023

I'm not sure of this README, I think there's maybe too much hyperlink. If you have suggestions that would be helpful. Let me ping @xmulligan as well maybe he has an opinion on READMEs? :)

Copy link
Contributor

@kkourt kkourt left a comment

Choose a reason for hiding this comment

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

I think it's an improvement over the current one, so LGTM!

@mtardy mtardy requested a review from xmulligan September 5, 2023 15:50
@jrfastab jrfastab merged commit 179a84c into main Sep 11, 2023
@jrfastab jrfastab deleted the pr/mtardy/readme-simplification branch September 11, 2023 21:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants