-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
doc: TF-M: add TF-M docset wrapper #20809
base: main
Are you sure you want to change the base?
Conversation
CI InformationTo view the history of this post, clich the 'edited' button above Inputs:Sources:more detailsGithub labels
List of changed files detected by CI (0)
Outputs:ToolchainVersion: Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped;
|
1bd58f3
to
01e3fe5
Compare
e1ad76d
to
d873855
Compare
|
33368f2
to
9bfc47a
Compare
@gmarull , can you have a look and see what is wrong? I've run out of ideas... |
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.
the problem is that you're adding a new main page that does not reference any of the others, and e.g. old index becomes "orphan". So the "best" you can do is to create a toctree in the wrapper that includes the old main page, but, this will break the original toctree and likely make things worse. Alternatively, you can maintain a noup
in our fork of TF-M...
9bfc47a
to
aaf9482
Compare
You can find the documentation preview for this PR here. |
9f0eb98
to
b3ee343
Compare
Added wrapper.rst and links.txt for the TF-M docset. NCSDK-32253. Signed-off-by: Grzegorz Ferenc <Grzegorz.Ferenc@nordicsemi.no>
b3ee343
to
ffc820f
Compare
Added wrapper.rst and links.txt for the TF-M docset. NCSDK-32253.
Wrapper preview at https://ncsdoc.z6.web.core.windows.net/PR-20809/tfm/wrapper.html -- the current index won't be replaced until this PR is merged due to server cache.
Next PR will add more information to the wrapper and the links.txt file.