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

Update docs README with new contribution workflow #82

Closed
wants to merge 0 commits into from

Conversation

banchan86
Copy link

This PR updates the instructions in README.md to reflect the most recent changes to the workflow for setting up and editing Bonsai docs.

  • Updates new contribution workflow.
  • Updates contributor style guide to reflect changes in operator and properties formatting.
  • Updates Bonsai workflow section to account for the new automatic SVG export with docfx-tools submodule.
  • Adds section on tutorial/examples submodule.
  • Adds section on including individual operator articles and overwriting API docs.
  • Adds section on setting up a new docfx website.
  • Adds sections on docs related workflows (testing unpublished packages, publishing to github packages).
  • Adds section on article and TOC organization.
  • Adds section on recommended article elements (alerts, mermaid diagrams, pipe tables).

An online version of this PR is available at https://github.com/banchan86/bonsai_docs_test/

@glopesdev glopesdev self-requested a review April 1, 2024 15:43
@glopesdev glopesdev changed the title Update README.md Update docs README with new contribution workflow Apr 22, 2024
README.md Outdated
- .github folder - contains a workflows/docs.yml file that is a Github Actions workflow recipe.
- Make sure that it is updated to the latest version and change the package name parameters.

- .bonsai folder - files necessary to build a Bonsai environment on Github Actions.
Copy link
Member

Choose a reason for hiding this comment

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

We might want to consider mentioning here how to build the solution in order to export the example workflows, and in that case also consider how to automate the msbuild part into the powershell module containing the export image functions.

README.md Outdated
- .gitignore file - this needs to be updated to ignore workflow files (i.e. .bonsai packages env).
```

7) Copy over these folders/files from a repo that has been recently updated into the "docs" folder.
Copy link
Member

Choose a reason for hiding this comment

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

We should maybe include these as part of docfx-tools or some other template repo.

README.md Outdated
href: articles/
- name: Reference
href: api/
- name: Tutorials
Copy link
Member

Choose a reason for hiding this comment

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

We might want to consider marking the Tutorials section as optional for other package documentation, to make things simpler as a common reference.

README.md Outdated
2) Create a `Network-Inference.md` article and place it in the `docs\articles` folder. In the markdown file, include a reference to the individual operator.md file.

```yml
[!include[Title](~/apidoc/Bonsai_Sleap_PredictPoses.md)]
Copy link
Member

Choose a reason for hiding this comment

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

Review format of relative / local URLs to make sure we have a solution that works for both local and remote builds.

README.md Outdated
3) Commit your edits and push to the `main` branch of your repo fork.
4) Under the `Actions` tab of your github repo, trigger the `Build docs` workflow manually with the `Run workflow` button. This will build the docs site on the `gh-pages` branch.
5) Once the `Build docs` workflow has been completed, the `pages-build-deployment` workflow will run and publish your forked repo automatically.
7) The URL for the site can be found in your `Pages` section of your repo settings.
Copy link
Member

Choose a reason for hiding this comment

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

It might be worth mentioning as a note the importance of matching the publishing URL with the PackageProjectUrl attribute specified in the .csproj files. This is where the Bonsai editor will look for the xrefmap.yml file to retrieve URLs for operators.

@glopesdev
Copy link
Member

@banchan86 I was going over the guide again, and was wondering whether this should be part of the docs README, or instead actually be an article inside the main bonsai docs, perhaps part of a larger guide on how to create new packages. Then in the README.md of this repo we could still point to this other article in case people drop by looking for guidance on how to create docs. Let's discuss better at the next meeting.

@banchan86
Copy link
Author

Oops accidentally closed this because I was rearranging my repo workspace, but I plan to reopen this with the changes addressed

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.

2 participants