Skip to content

Conversation

amanji
Copy link
Contributor

@amanji amanji commented Jan 16, 2023

This PR adds a doc site (built with Docusaurus) to ACA-Py. RTD files have been moved into a subfolder of docs and Sphinx-generated .rst files are converted to markdown to be included in docs.

amanji added 15 commits January 16, 2023 08:52
Signed-off-by: Akiff Manji <amanji@petridish.dev>
Signed-off-by: Akiff Manji <amanji@petridish.dev>
Signed-off-by: Akiff Manji <amanji@petridish.dev>
Signed-off-by: Akiff Manji <amanji@petridish.dev>
Signed-off-by: Akiff Manji <amanji@petridish.dev>
Signed-off-by: Akiff Manji <amanji@petridish.dev>
Signed-off-by: Akiff Manji <amanji@petridish.dev>
Signed-off-by: Akiff Manji <amanji@petridish.dev>
…aurus

Signed-off-by: Akiff Manji <amanji@petridish.dev>
Signed-off-by: Akiff Manji <amanji@petridish.dev>
Signed-off-by: Akiff Manji <amanji@petridish.dev>
Signed-off-by: Akiff Manji <amanji@petridish.dev>
Signed-off-by: Akiff Manji <amanji@petridish.dev>
Signed-off-by: Akiff Manji <amanji@petridish.dev>
Signed-off-by: Akiff Manji <amanji@petridish.dev>
Signed-off-by: Akiff Manji <amanji@petridish.dev>
Signed-off-by: Akiff Manji <amanji@petridish.dev>
@swcurran
Copy link
Contributor

I don't think we can merge this as the move of the RTD docs will break the RTD site we have. We need to rethink how we are adding generated site support to the repo.

Signed-off-by: Akiff Manji <amanji@petridish.dev>
Signed-off-by: Akiff Manji <amanji@petridish.dev>
@swcurran
Copy link
Contributor

@amanji -- what is needed to get this merged? Here are the questions/confirmations I have:

  1. The RTD generated folder is currently manually maintained -- e.g. the files in the folder are produced locally (via a couple of commands) and then a PR created with any updates. In theory that could be done via a GHA, but in my experience, with that type of thing, it's a bit of pain, and might make the current ReadTheDocs generation difficult. I suggest that we leave it as is for now, at least until we deprecate and eliminate the current readthedocs site.

  2. You need to clean up the problematic docstrings in the code -- the use of <>s. At some point, we want to add a check for that in a PR test, but not needed yet.

  3. Why are the "assets" eliminated? Those are currently being referenced in the MD files in the root folder, I believe. I believe they are needed and if you move them, you need to update the MDs that reference them, so they don't break.

Are there any other things needed? Once we've merged this, we can launch the gh-pages site and iterate on it, right?

@amanji
Copy link
Contributor Author

amanji commented Jan 19, 2023

  • The Docusaurus flow should, in theory, not break the current RTD, so I'll leave those generated .rst files as is and we can update them manually for safety.
  • Working on a Remark/Rehype plugin to fix the problematic markdown
    • This may take a bit more time so will use an external link to get things moving along
  • Assets were accidentally removed, will add those back and update the links as needed
  • Adding an example of a version dropdown
  • Re-add the homepage components back (will leave out the images for now)

Once the PR is merged we simply need to enable GH Pages to be deployed from a branch (this will be in the repo settings) the branch should be set to gh-pages and from there it's all automatic.

Signed-off-by: Akiff Manji <amanji@petridish.dev>
Signed-off-by: Akiff Manji <amanji@petridish.dev>
Signed-off-by: Akiff Manji <amanji@petridish.dev>
@swcurran
Copy link
Contributor

I accidentally ran yarn at the root level instead of the docs level. Is it worth putting the .gitignore entries at a higher level? No problem if inappropriate, but wondered.

Other than that, looks good. I will amp up the how to run locally documentation -- it's bare, but I got through it. Had some fun on Ubuntu with node versions, what to upgrade to and so on. Then had finding authoritative information on installing/updating and what versions of node both worked and ran on my system. Finally, had a weird error about number of "watchers" parameter in my Linux config that I had to fix. Fun stuff.

Let me know if you think the gitignore is worth doing and then I will merge when I have a next chance to look at it.

amanji added 18 commits February 7, 2023 11:27
This reverts commit ab8781f.
This reverts commit 529375a.
This reverts commit ae33d26.
This reverts commit baf0464.
Signed-off-by: Akiff Manji <amanji@petridish.dev>
Signed-off-by: Akiff Manji <amanji@petridish.dev>
Signed-off-by: Akiff Manji <amanji@petridish.dev>
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

Signed-off-by: Akiff Manji <amanji@petridish.dev>
Signed-off-by: Akiff Manji <amanji@petridish.dev>
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@amanji
Copy link
Contributor Author

amanji commented Feb 23, 2023

Will re-open once local testing complete

@amanji amanji closed this Feb 23, 2023
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.

3 participants