You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I think we solved this, right?
SEEKs github action will build the docs and push them to this repo. This repo's github action will build the generator and visualize docs, as well as create a new docker image. As soon as that docker image is built it update's our Azure demo site and all documentation can be viewed here: http://neuroseek.azurewebsites.net/docs/
One thing that we could discuss is perhaps moving docs hosting to a diff branch? And then that way when we build prs on this repo which get merged to master we don't need to sync in the doc changes (pushed by seek)?
E.g. currently:
Seekpipeline say makes a change in docs and it pushes here for hosting
We make a change here in a PR branch
We need to git sync master branch again cuz docs changed.
Alternatively
Seekpipeline makes changes. It pushed doc changes to 'docs' branch here to be re rendered.
We make changes here in a PR branch
When we merge into master, it gets pushed to 'docs' branch and is re rendered.
But then this can bring up complications:
How do we make the docs appear in CI for every PR here? Possibly not even a problem tho depending on how docs build and deployment occurs.
Any other challenges?
If it's rather challenging overhaul might not be necessary to tackle now.
Summary
So was thinking where should we host a documentation page and should they be separate on repos and link to each other, or on one repo?
Possible Solutions
Option 1:
SEEK has its own doc page on how to run workflows that then connect to a visualization engine
ReconstructionVisualizer then has its own docs page that links to SEEK to specify what data inputs it wants.
Option 2:
Aggregated docs page here?
Then these should be ideally rendered during CI for every PR and master branch commit.
The text was updated successfully, but these errors were encountered: