-
Notifications
You must be signed in to change notification settings - Fork 123
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
xilem: Docs updating #818
base: main
Are you sure you want to change the base?
xilem: Docs updating #818
Conversation
0f4d725
to
f6e1f33
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for providing a foundation here. The CI needs to be green to land this.
Thank you for comments. Before I left some red alerts for purpose: I want to do some more work inside this PR. So, now I am going to look at Masonry docs and then will make some inserts and updates. |
17cf29c
to
0173adb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for doing this - the content for the views look good.
As mentioned, the new README content needs to be added to the "formatting" CI job.
I'm willing to make the corresponding changes to the README myself, outside of this PR, if you prefer.
df1e9e0
to
9796c5b
Compare
9796c5b
to
47d0077
Compare
I've converted to draft based on your Draft in the top-level body of the PR. When you're ready for this to be reviewed/land, please remove it from draft (or one of us can finish it off if you need) |
I think I've finished. Maybe I'll do the rest later, maybe in the next PR (if nobody does it in the near future). |
Draft
This PR is the proposal for xilem docs starting page. But I think I will be able to make some more commits to improve documentation.