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

Documentation Restructure #1337

Merged
merged 6 commits into from Sep 12, 2022
Merged

Documentation Restructure #1337

merged 6 commits into from Sep 12, 2022

Conversation

donn
Copy link
Collaborator

@donn donn commented Sep 8, 2022

~ Documentation all moved under docs/source, arranged hierarchically according to the table of contents
~ Klayout changed to KLayout in all logging messages
~ Readme rewritten to just be concise, parts of it isolated into standalone documentation
~ RTD builds no longer use conda (saves some time)
~ Fixed all broken links


This isn't strictly part of this PR, but I also activated RTD for pull requests. You can press the details button down below to preview everything I did:

image

~ Documentation all moved under `docs/source`, arranged hierarchically according to the table of contents
~ Readme rewritten, parts of it isolated into standalone documentation
@donn donn force-pushed the new_readme branch 2 times, most recently from 7f67f2a to 429a400 Compare September 8, 2022 06:57
@donn donn marked this pull request as ready for review September 8, 2022 07:07
@donn donn requested a review from kareefardi September 8, 2022 07:07
@kareefardi
Copy link
Collaborator

Klayout changed to KLayout in all logging messages.

@donn donn merged commit cbb9e6b into The-OpenROAD-Project:master Sep 12, 2022
@donn donn deleted the new_readme branch September 12, 2022 12:49
@donn donn mentioned this pull request Sep 12, 2022
@armleo
Copy link
Contributor

armleo commented Sep 12, 2022

"Installation, the long version" leads to 404 @donn

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.

None yet

3 participants