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

doc: security: Don't document how to flash a TF-M #11960

Merged
merged 1 commit into from
Aug 7, 2023

Conversation

SebastianBoe
Copy link
Contributor

Don't document how to flash TF-M as it is the same as for any NCS build.

This documentation is expensive to maintain as it is referencing development environment specifics that are unrelated to TF-M and subject to change.

Don't document how to flash TF-M as it is the same as for any NCS
build.

This documentation is expensive to maintain as it is referencing
development environment specifics that are unrelated to TF-M and
subject to change.

Signed-off-by: Sebastian Bøe <sebastian.boe@nordicsemi.no>
@github-actions github-actions bot added changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. doc-required PR must not be merged without tech writer approval. labels Aug 4, 2023
@SebastianBoe SebastianBoe requested review from Vge0rge and removed request for Vge0rge August 4, 2023 11:14
@NordicBuilder
Copy link
Contributor

Test specification

CI/Jenkins/NRF

  • Skipped

CI/Jenkins/integration

  • Skipped

Note: This message is automatically posted and updated by the CI

@NordicBuilder
Copy link
Contributor

You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds.

Note: This comment is automatically posted by the Documentation Publishing GitHub Action.

Copy link
Contributor

@mswarowsky mswarowsky left a comment

Choose a reason for hiding this comment

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

Fine for me to remove that part

@nordicjm nordicjm merged commit c83856e into nrfconnect:main Aug 7, 2023
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. doc-required PR must not be merged without tech writer approval.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants